Skip to content

replay: Also use revision parameter when calling client.info

Antoine Lambert requested to merge generated-differential-D7047-source into generated-differential-D7047-target

It exist cases where the client.info command will fail if both peg_revision and revision parameters are not provided.

Python 3.9.2 (default, Feb 28 2021, 17:03:44)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from subvertpy import SubversionException, client, properties
>>> from subvertpy.ra import Auth, RemoteAccess, get_username_provider
>>> auth = Auth([get_username_provider()])
>>> client = client.Client(auth=auth)
>>> client.info("http://svn.code.sf.net/p/phpbbproject/svn/trunk", peg_revision=260)
Traceback (most recent call last):   
  File "<stdin>", line 1, in <module>
subvertpy.SubversionException: ("Unable to find repository location for 'http://svn.code.sf.net/p/phpbbproject/svn/trunk' in revision 318", 195012)
>>> client.info("http://svn.code.sf.net/p/phpbbproject/svn/trunk", revision=260)
Traceback (most recent call last):   
  File "<stdin>", line 1, in <module>
subvertpy.SubversionException: ("Unable to find repository location for 'http://svn.code.sf.net/p/phpbbproject/svn/trunk' in revision 260", 195012)
>>> client.info("http://svn.code.sf.net/p/phpbbproject/svn/trunk", peg_revision=260, revision=260)
{'trunk': <client.Info object at 0x7f18d2a42eb0>}

Fixes SWH-LOADER-SVN-5K

Related to #611

Migrated from D7047 (view on Phabricator)

Merge request reports