aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Frysinger <vapier@google.com>2023-09-06 21:35:07 -0400
committerMike Frysinger <vapier@google.com>2023-09-06 21:36:35 -0400
commit99cf377b6de9b16e160fee796139d6f63d31b9ea (patch)
tree8f26ff335d180a6a5070cd0f348322205bf6472f
parent904ef5628355fefe262ea1cfc262cca2193bc120 (diff)
downloadrepohooks-99cf377b6de9b16e160fee796139d6f63d31b9ea.tar.gz
This is a named tuple, so we need to replace the value, not set it. Bug: 299373007 Test: None Change-Id: I86eb65c484878c11c2f02b6a0c6212943895f922
-rwxr-xr-xpre-upload.py6
1 files changed, 2 insertions, 4 deletions
diff --git a/pre-upload.py b/pre-upload.py
index 3eece2d..18bf11f 100755
--- a/pre-upload.py
+++ b/pre-upload.py
@@ -359,8 +359,7 @@ def _run_project_hooks_in_cwd(
config = _get_project_config(from_git)
except rh.config.ValidationError as e:
output.error('Loading config files', str(e))
- ret.internal_failure = True
- return ret
+ return ret._replace(internal_failure=True)
# If the repo has no pre-upload hooks enabled, then just return.
hooks = list(config.callable_hooks())
@@ -374,8 +373,7 @@ def _run_project_hooks_in_cwd(
except rh.utils.CalledProcessError as e:
output.error('Upstream remote/tracking branch lookup',
f'{e}\nDid you run repo start? Is your HEAD detached?')
- ret.internal_failure = True
- return ret
+ return ret._replace(internal_failure=True)
project = rh.Project(name=project_name, dir=proj_dir)
rel_proj_dir = os.path.relpath(proj_dir, rh.git.find_repo_root())