From 478c9e4851eff5feb74dd2f8a14143bec569e1a8 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Wed, 1 Jul 2015 16:35:59 -0400 Subject: Revert "Move headers used by headers in include/ to include/private." This reverts commit 928e16565f3e69cd6661e9ad300ac17e3f33c0c8. BUG=skia: Review URL: https://codereview.chromium.org/1213093004. --- PRESUBMIT.py | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'PRESUBMIT.py') diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 04ed2e0607..5599c316c3 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -279,10 +279,8 @@ 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, but not include/private. - if (file_ext == '.h' and - 'include' == file_path.split(os.path.sep)[0] and - 'private' not in file_path): + # include dir. + if file_ext == '.h' and 'include' == file_path.split(os.path.sep)[0]: requires_owner_check = True if not requires_owner_check: @@ -330,12 +328,8 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): if not lgtm_from_owner: results.append( output_api.PresubmitError( - "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))) + 'Since the CL is editing public API, you must have an LGTM from ' + 'one of: %s' % str(PUBLIC_API_OWNERS))) return results -- cgit v1.2.3