tree ecff9afc7275d0027cb83425c097645d36fed06a
parent 23c62e9f8087539c08cbee2c477853849aa69f3b
author Brian Norris <briannorris@chromium.org> 1544563469 -0800
committer chrome-bot <chrome-bot@chromium.org> 1544572595 -0800

pre-upload: fix a few style/compatibility issues

A few review notes came after CL:1336039 landed.

 * os.path.join: might as well use this everywhere we're constructing
   paths out of _get_hooks_dir()
 * it's a little clearer to name the non-msg HookFailure() args

BUG=none
TEST=unit tests; run a few good/bad commits through .../pre-upload.py

Change-Id: I401fe6c8bf02a3020122fa0875b2beb23345be27
Reviewed-on: https://chromium-review.googlesource.com/1372334
Commit-Ready: Brian Norris <briannorris@chromium.org>
Tested-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>