tree c2a4f3a64d21fea3ea7a9e47fca401bcc00725fc
parent c627874f1bba21f33406852c2425eeb4b577a599
author Stephen Boyd <swboyd@chromium.org> 1602745327 -0700
committer Commit Bot <commit-bot@chromium.org> 1602876991 +0000

pre-upload: Skip revbump check for profiles paths

If a CL combines a profile update (like make.defaults) for an overlay
with an ebuild change to the files/ directory a confusing error message
comes out from the repo upload step. For example, uploading CL:2446471
causes the check to complain that the ebuild needs a manual revbump when
it really doesn't. This is because the hook is checking all paths in the
commit, including ones for the overlay's 'make.defaults'. Skip out on
paths that have '/profiles/' in them so that the revbump logic doesn't
get confused and lead developers astray.

BUG=None
TEST=Try to upload CL:2446471 and see it doesn't complain anymore

Change-Id: I1cffc6592e81a28d922481a704fa4e1e699919b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/repohooks/+/2473371
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Stephen Boyd <swboyd@chromium.org>
Tested-by: Stephen Boyd <swboyd@chromium.org>