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>
1 file changed