diff options
author | mtklein <mtklein@chromium.org> | 2015-07-01 08:41:14 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-01 08:41:15 -0700 |
commit | a89f55198bdc58f0b6f6196907ab25a6afc1a661 (patch) | |
tree | a608ef4f5fe9fd724bd57ddade2b62c00bf60ae5 /PRESUBMIT.py | |
parent | bd929d95704d052a5c8477b881761eff03582487 (diff) |
Move headers used by headers in include/ to include/private.
Some of this is transitive, like SkRecords.h used by SkMiniRecorder.h
used by (public) SkPictureRecorder.h.
BUG=skia:
Review URL: https://codereview.chromium.org/1217293004
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 5599c316c3..04ed2e0607 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -279,8 +279,10 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): affected_file_path = affected_file.LocalPath() file_path, file_ext = os.path.splitext(affected_file_path) # We only care about files that end in .h and are under the top-level - # include dir. - if file_ext == '.h' and 'include' == file_path.split(os.path.sep)[0]: + # include dir, but not include/private. + if (file_ext == '.h' and + 'include' == file_path.split(os.path.sep)[0] and + 'private' not in file_path): requires_owner_check = True if not requires_owner_check: @@ -328,8 +330,12 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): if not lgtm_from_owner: results.append( output_api.PresubmitError( - 'Since the CL is editing public API, you must have an LGTM from ' - 'one of: %s' % str(PUBLIC_API_OWNERS))) + "If this CL adds to or changes Skia's public API, you need an LGTM " + "from any of %s. If this CL only removes from or doesn't change " + "Skia's public API, please add a short note to your CL saying so " + "and add one of those reviewers on a TBR= line. If you don't know " + "this CL affects Skia's public API, please treat it like it does." + % str(PUBLIC_API_OWNERS))) return results |