upload: Return correct tuple values in _ProcessResults

Incorrect tuple values were returned with http://go/grev/440221 -
instead of returning (Project, ReviewableBranch), _ProcessResults was
returning (int, ReviewableBranch).

R=jojwang@google.com

Bug: 376731172
Change-Id: I75205f42fd23f5ee6bd8d0c15b18066189b42bd9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/441121
Reviewed-by: Sam Saccone <samccone@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Tested-by: Josip Sokcevic <sokcevic@google.com>

project.py

@@ -3375,20 +3375,25 @@ class Project:
             setting = fp.read()
             assert setting.startswith("gitdir:")
             git_worktree_path = setting.split(":", 1)[1].strip()
+
+        # `gitdir` maybe be either relative or absolute depending on the
+        # behavior of the local copy of git, so only convert the path to
+        # relative if it needs to be converted.
+        if os.path.isabs(git_worktree_path):
             # Some platforms (e.g. Windows) won't let us update dotgit in situ
-        # because of file permissions.  Delete it and recreate it from scratch
+            # because of file permissions.  Delete it and recreate it from
-        # to avoid.
+            # scratch to avoid.
             platform_utils.remove(dotgit)
-        # Use relative path from checkout->worktree & maintain Unix line endings
+            # Use relative path from checkout->worktree & maintain Unix line
-        # on all OS's to match git behavior.
+            # endings on all OS's to match git behavior.
             with open(dotgit, "w", newline="\n") as fp:
                 print(
                     "gitdir:",
                     os.path.relpath(git_worktree_path, self.worktree),
                     file=fp,
                 )
-        # Use relative path from worktree->checkout & maintain Unix line endings
+            # Use relative path from worktree->checkout & maintain Unix line
-        # on all OS's to match git behavior.
+            # endings on all OS's to match git behavior.
             with open(
                 os.path.join(git_worktree_path, "gitdir"), "w", newline="\n"
             ) as fp:

subcmds/forall.py

@@ -298,7 +298,7 @@ without iterating through the remaining projects.
                 )
         except (KeyboardInterrupt, WorkerKeyboardInterrupt):
             # Catch KeyboardInterrupt raised inside and outside of workers
-            rc = rc or errno.EINTR
+            rc = errno.EINTR
         except Exception as e:
             # Catch any other exceptions raised
             logger.error(
@@ -306,7 +306,7 @@ without iterating through the remaining projects.
                 type(e).__name__,
                 e,
             )
-            rc = rc or getattr(e, "errno", 1)
+            rc = getattr(e, "errno", 1)
         if rc != 0:
             sys.exit(rc)
 

subcmds/upload.py

@@ -737,8 +737,8 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
             pending = []
             for result in results:
                 project_idx, avail = result
-                if avail is None:
                 project = projects[project_idx]
+                if avail is None:
                     logger.error(
                         'repo: error: %s: Unable to upload branch "%s". '
                         "You might be able to fix the branch by running:\n"
@@ -748,7 +748,7 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
                         project.manifest.branch,
                     )
                 elif avail:
-                    pending.append(result)
+                    pending.append((project, avail))
             return pending
 
         with self.ParallelContext():