"Ben Keene via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> @@ -2337,12 +2405,15 @@ def run(self, args):
> sys.exit("number of commits (%d) must match number of shelved changelist (%d)" %
> (len(commits), num_shelves))
>
> - hooks_path = gitConfig("core.hooksPath")
> - if len(hooks_path) <= 0:
> - hooks_path = os.path.join(os.environ.get("GIT_DIR", ".git"), "hooks")
> -
> - hook_file = os.path.join(hooks_path, "p4-pre-submit")
> - if os.path.isfile(hook_file) and os.access(hook_file, os.X_OK) and subprocess.call([hook_file]) != 0:
> + try:
> + if not run_git_hook("p4-pre-submit"):
> + print("\nThe p4-pre-submit hook failed, aborting the submit.\n\nYou can skip " \
> + "this pre-submission check by adding\nthe command line option '--no-verify', " \
> + "however,\nthis will also skip the p4-changelist hook as well.")
> + sys.exit(1)
> + except Exception as e:
> + print("\nThe p4-pre-submit hook failed, aborting the submit.\n\nThe hook failed "\
> + "with the error '{0}'".format(e.message) )
> sys.exit(1)
If I am reading the patches correctly, we do not have the
"--no-verify" option at this point in the series. Shouldn't this
step limit itself to replace the removed lines we see above
with just a singlie liner:
if not run_git_hook("p4-pre-submit"):
sys.exit(1)
instead?
Also, af the end of these 6-patch series, I do not see a change to
guard this part of the code with "if not self.no_verify" like the
previous round did.
