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

sync: PersistentTransport call parent init

2023-10-25T09:06:23+00:00

Found via pylint: W0231: __init__ method from base class 'Transport' is not called (super-init-not-called) Just fixed for code correctness and to avoid potential future bugs. Change-Id: Ie1e723c2afe65d026d70ac01a16ee7a40c149834 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390676 Reviewed-by: Mike Frysinger Tested-by: Daniel Kutik Commit-Queue: Daniel Kutik

sync: TeeStringIO write should return int

2023-10-24T19:29:57+00:00

Change-Id: I211776a493cad4b005c6e201833e9700def2feb9 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390657 Reviewed-by: Mike Frysinger Commit-Queue: Daniel Kutik Tested-by: Daniel Kutik

sync: Fix tracking of broken links

2023-10-24T18:49:20+00:00

Change-Id: Ice4f4cc745cbac59f356bd4ce1124b6162894e61 Bug: b/113935847 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390434 Tested-by: Jason Chang Reviewed-by: Josip Sokcevic Reviewed-by: Mike Frysinger Commit-Queue: Jason Chang

project: Speculative fix for project corruption

2023-10-23T16:13:02+00:00

When a new shared project is added to manifest, there's a short window where objects can be deleted that are used by other projects. To close that window, set preciousObjects during git init. For non-shared projects, repo should correct the state in the same execution instance. Bug: 288102993 Change-Id: I366f524535ac58c820d51a88599ae2108df9ab48 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390234 Commit-Queue: Josip Sokcevic Tested-by: Josip Sokcevic Reviewed-by: Mike Frysinger

cleanup: convert exceptions to OSError

2023-10-21T00:56:10+00:00

In Python 3, these exceptions were merged into OSError, so switch everything over to that. Change-Id: If876a28b692de5aa5c62a3bdc8c000793ce52c63 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390376 Reviewed-by: Aravind Vasudevan Commit-Queue: Mike Frysinger Tested-by: Mike Frysinger

cleanup: delete redundant "r" open mode

2023-10-21T00:55:33+00:00

Change-Id: I86ebb8c5a9dc3752e8a25f4b11b64c5be3a6429e Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390375 Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger Reviewed-by: Aravind Vasudevan

cleanup: use new dict & set generator styles

2023-10-21T00:55:01+00:00

Change-Id: Ie34ac33ada7855945c77238da3ce644f8a9f8306 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390374 Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger Reviewed-by: Aravind Vasudevan

manifest_xml: Fix empty project list when DOCTYPE is present

2023-10-20T18:22:59+00:00

When parsing the manifest XML, the code looks for a top level DOM node named "manifest". However, it doesn't check that it's an element type node so if there is also an XML document type declaration node present (which has the same name as the root element) then it selects the wrong node and hence you end up with no projects defined at all. Change-Id: I8d101caffbbc2a06e56136ff21302e3f09cfc96b Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390357 Reviewed-by: Mike Frysinger Tested-by: Chris Allen Commit-Queue: Chris Allen

cleanup: leverage yield from in more places

2023-10-20T17:33:03+00:00

Change-Id: I4f9cb27d89241d3738486764817b51981444a903 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390274 Reviewed-by: Aravind Vasudevan Commit-Queue: Mike Frysinger Tested-by: Mike Frysinger

git_config: GetBoolean should return bool

2023-10-20T16:41:58+00:00

Test: tox Change-Id: Ifc0dc089deef5a3b396d889c9ebfcf8d4f007bf2 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390360 Tested-by: Daniel Kutik Reviewed-by: Mike Frysinger Commit-Queue: Daniel Kutik