tools/git-repo.git, branch v2.31

tests: drop old unittest.main logic

2022-12-08T17:17:30+00:00

We use pytest now which doesn't need this boilerplate. Change-Id: Ib71d90b3f1669897814ee768927b5b595ca8d789 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/354355 Reviewed-by: Josip Sokcevic Tested-by: Mike Frysinger

sync: fix manifest sync-j handling

2022-12-08T15:06:24+00:00

Since --jobs defaults to 0, not None, we never pull the value out of the manifest. Treat values of 0 and None the same to fix. Bug: http://b/239712300 Bug: http://b/260908907 Change-Id: I9b1026682072366616825fd72f90bd90c10a252f Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/354254 Tested-by: Mike Frysinger Reviewed-by: Josip Sokcevic Reviewed-by: Sam Saccone

sync: Fix undefined variable in _FetchOne

2022-12-08T06:29:00+00:00

If syncing in _FetchOne fails with GitError, sync_result does not get set. There's already a separate local variable for success; do the same for remote_fetched instead of referring to the conditionally defined named tuple. This bug is originally caused by a combination of ad8aa697 "sync: only print error.GitError, don't raise that exception." and 1eddca84 "sync: use namedtuples for internal return values". Change-Id: I0f9dbafb97f8268044e5a56a6f92cf29bc23ca6a Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/354176 Tested-by: Karsten Tausche Reviewed-by: LaMont Jones

sync: finish marking REPO_AUTO_GC=1 as deprecated.

2022-12-05T18:11:24+00:00

The wrong revision of the change was submitted as d793553804c76677444709ebefd70f6e01c29525. Change-Id: I6f3e4993cf40c30ccf0d69020177db8fe5f76b8c Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/353934 Tested-by: LaMont Jones Reviewed-by: Sam Saccone

test_manifest_config_properties: use assertEqual

2022-12-05T13:09:09+00:00

The method assertEquals is an deprecated alias for assertEqual. See: https://docs.python.org/3/library/unittest.html#deprecated-aliases Change-Id: Id94ba6d6055bdc18b87c53e8729902bb278855aa Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/354035 Reviewed-by: Mike Frysinger Tested-by: Daniel Kutik

sync: cleanup output when not doing GC

2022-12-02T22:50:23+00:00

Do not use a progress bar when not doing GC, and restrict activity in that case to only repairing preciousObject state. This also includes additional cleanup based on review comments from previous changes. Change-Id: I48581c9d25da358bc7ae15f40e98d55bec142331 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/353514 Tested-by: LaMont Jones Reviewed-by: Mike Frysinger

Revert "sync: save any cruft after calling git gc."

2022-12-02T22:40:06+00:00

This bug-cacher related code is no longer needed. This reverts commit 891e8f72ce3551a19c377456574bbfbeac5c8b8e. Change-Id: Ia94a2690ff149427fdcafacd39f5008cd60827d5 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/353774 Reviewed-by: Mike Frysinger Reviewed-by: Sam Saccone Tested-by: LaMont Jones

sync: mark REPO_AUTO_GC=1 as deprecated.

2022-12-02T22:33:11+00:00

REPO_AUTO_GC was introduced as a way for users to restore the previous default behavior, since the default changed at the same time as the option was added. As such, it should be marked as deprecated, and removed entirely in a future release. Change-Id: Ib73d98fbea693e7057cc4587928c225a9e4beab2 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/353734 Reviewed-by: Sam Saccone Tested-by: LaMont Jones Reviewed-by: Mike Frysinger

Fix ManifestProject.partial_clone_exclude property.

2022-12-02T14:57:56+00:00

Bug: b/256358360 Change-Id: Ic6e3a049aa38827123d0324c8b14157562c5986e Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/353574 Reviewed-by: Mike Frysinger Tested-by: Joanna Wang Reviewed-by: Josip Sokcevic

release-process: update to use ./release/sign-tag.py

2022-12-01T00:04:24+00:00

We have a helper script for signing releases now, so point the docs to that rather than the multiple manual steps. Change-Id: I309e883dbce1894650e31682d9975cf0d6bdeca3 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/352834 Tested-by: Mike Frysinger Reviewed-by: Josip Sokcevic