diff options
author | Mike Frysinger <vapier@google.com> | 2023-09-06 21:35:07 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2023-09-06 21:36:35 -0400 |
commit | 99cf377b6de9b16e160fee796139d6f63d31b9ea (patch) | |
tree | 8f26ff335d180a6a5070cd0f348322205bf6472f | |
parent | 904ef5628355fefe262ea1cfc262cca2193bc120 (diff) | |
download | repohooks-99cf377b6de9b16e160fee796139d6f63d31b9ea.tar.gz |
pre-upload: fix internal_failure settingub-automotive-master-20231102ub-automotive-master-20231011android-vts-14.0_r2android-cts-14.0_r2androidx-wear-wear-tooling-preview-releaseandroidx-core-performance-release
This is a named tuple, so we need to replace the value, not set it.
Bug: 299373007
Test: None
Change-Id: I86eb65c484878c11c2f02b6a0c6212943895f922
-rwxr-xr-x | pre-upload.py | 6 |
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()) |