Mirror of gerrit.googlesource.com/git-repo https://git.enea.com/cgit/tools/git-repo.git/atom?h=v2.12 2021-01-21T19:41:52+00:00 2021-01-21T19:41:52+00:00 Raman Tenneti rtenneti@google.com 2021-01-15T03:17:50+00:00 urn:sha1:6a872c9daeec58dda61786e6f65591330709f6ad Added "--use-superporject" option to sync.py to fetch project SHAs from superproject. If there are any missing projects in superprojects, it prints the missing entries and exits. If there are no missing entries, it will use SHAs from superproject to fetch the projects from git. Tested the code with the following commands. $ ./run_tests tests/test_manifest_xml.py $ ./run_tests -v tests/test_git_superproject.py $ ./run_tests -v Tested the sync code by copying all the repo changes into my Android AOSP checkout and adding <superporject> tag to default.xml. With local modification to the code to print the status, .../WORKING_DIRECTORY$ repo sync --use-superproject repo: executing 'git clone' url: sso://android/platform/superproject repo: executing 'git ls-tree' Success: [] Bug: https://crbug.com/gerrit/13709 Tested-by: Raman Tenneti <rtenneti@google.com> Change-Id: Id18665992428dd684c04b0e0b3a52f46316873a0 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/293822 Reviewed-by: Mike Frysinger <vapier@google.com> 2021-01-06T18:53:05+00:00 Mike Frysinger vapier@google.com 2021-01-01T13:42:31+00:00 urn:sha1:784ccfc040dc8efa1a64d3c7d4070b66beb15d08 We're committed to Python 3 at this point, so clean up boilerplate. Bug: https://crbug.com/gerrit/10418 Change-Id: Ib1719ba2eb65c53b94881a1a1bf203ddfcaaafed Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/292382 Reviewed-by: Chris Mcdonald <cjmcdonald@google.com> Tested-by: Mike Frysinger <vapier@google.com> 2020-02-12T06:36:40+00:00 David Pursehouse dpursehouse@collab.net 2020-02-12T06:20:19+00:00 urn:sha1:819827a42ddb364f98c3a1a7eae2536dc54bc4cc - E301 expected 1 blank line - E302 expected 2 blank lines - E303 too many blank lines - E305 expected 2 blank lines after class or function definition - E306 expected 1 blank line before a nested definition Fixed automatically with autopep8: git ls-files | grep py$ | xargs autopep8 --in-place \ --select E301,E302,E303,E305,E306 Manually fix issues in project.py caused by misuse of block comments. Change-Id: Iee840fcaff48aae504ddac9c3e76d2acd484f6a9 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254599 Reviewed-by: Mike Frysinger <vapier@google.com> Tested-by: David Pursehouse <dpursehouse@collab.net> 2020-02-04T20:34:01+00:00 Mike Frysinger vapier@google.com 2019-08-01T03:32:58+00:00 urn:sha1:04122b7261319dae3abcaf0eb63af7ed937dc463 Reject paths in <copyfile> & <linkfile> that point outside of their respective scopes. This validates paths while parsing the manifest as this should be quick & cheap: we don't access the filesystem as this code runs before we've synced. Bug: https://crbug.com/gerrit/11218 Change-Id: I8e17bb91f3f5b905a9d76391b29fbab4cb77aa58 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/232932 Tested-by: Mike Frysinger <vapier@google.com> Reviewed-by: Mike Frysinger <vapier@google.com> Reviewed-by: Michael Mortensen <mmortensen@google.com> 2019-06-13T14:30:52+00:00 Mike Frysinger vapier@google.com 2019-06-13T06:30:51+00:00 urn:sha1:f601376e13e6beca7ab8d660e00d6c14714f305f There's no reason to support any other encoding in these files. This only affects the files themselves and not streams they open. Bug: https://crbug.com/gerrit/10418 Change-Id: I053cb40cd3666ce5c8a0689b9dd938f24ca765bf 2015-06-04T00:21:16+00:00 Anthony King anthonydking@slimroms.net 2015-06-03T15:59:18+00:00 urn:sha1:be4456cf249f29371eef41bed4e15f6d3c1a2cc0 Change-Id: I09c47024ef54c360ea3c15c5d4f169e13444e412 2014-03-11T05:33:43+00:00 Dan Sandler dsandler@android.com 2014-03-09T17:20:02+00:00 urn:sha1:53e902a19b0b80e07ac55966d13c5c84c5b0e8ce The old "manifest required for this command -- please run init" is replaced by a more helpful message that lists the command repo was trying to execute (with arguments) as well as the str() of the NoManifestException. For example: > error: in `sync`: [Errno 2] No such file or directory: > 'path/to/.repo/manifests/.git/HEAD' > error: manifest missing or unreadable -- please run init Other failure points in basic command parsing and dispatch are more clearly explained in the same fashion. Change-Id: I6212e5c648bc5d57e27145d55a5391ca565e4149 2012-11-16T02:50:11+00:00 Conley Owens cco3@android.com 2012-11-16T01:33:11+00:00 urn:sha1:75ee0570da09abb1d2bbefe0d25f0560727e6b71 When a command (eg, `repo forall`) expects the manifest project to exist, but there is no manifest, an IOException gets raised. This change defines a new Exception type to be raised in these cases and raises it when project.py fails to read the manifest. Change-Id: Iac576c293a37f7d8f60cd4f6aa95b2c97f9e7957 2012-10-22T03:30:14+00:00 David Pursehouse david.pursehouse@sonymobile.com 2012-10-11T07:44:48+00:00 urn:sha1:5c6eeac8f0350fd6b14cf226ffcff655f1dd9582 Fixing more issues found with pylint. Some that were supposed to have been fixed in the previous sweep (Ie0db839e) but were missed: C0321: More than one statement on a single line W0622: Redefining built-in 'name' And some more: W0631: Using possibly undefined loop variable 'name' W0223: Method 'name' is abstract in class 'name' but is not overridden W0231: __init__ method from base class 'name' is not called Change-Id: Ie119183708609d6279e973057a385fde864230c3 2012-10-09T10:45:30+00:00 David Pursehouse david.pursehouse@sonymobile.com 2012-09-24T03:15:13+00:00 urn:sha1:8a68ff96057ec58e524a3e41a2d8dca7b5d016bc Fix the following issues reported by pylint: C0321: More than one statement on a single line W0622: Redefining built-in 'name' W0612: Unused variable 'name' W0613: Unused argument 'name' W0102: Dangerous default value 'value' as argument W0105: String statement has no effect Also fixed a few cases of inconsistent indentation. Change-Id: Ie0db839e7c57d576cff12d8c055fe87030d00744