From 32ce6d81c435e28750acdb839e2910f30610e16f Mon Sep 17 00:00:00 2001 From: "dcaiafa@google.com" Date: Thu, 6 Mar 2014 18:55:44 +0000 Subject: Revert 255357 "Change the user action file format from .txt to ...." Believed to be behind the failures in the Linux builder: http://build.chromium.org/p/chromium/builders/Linux/builds/47996 Errors in the log: .../actions/print_style.py: Has executable bit but not shebang or ELF header .../histograms/print_style.py: Has executable bit but not shebang or ELF header Refers to files in the CL. > Change the user action file format from .txt to .xml. > > In this way, more information can be added (currently added 'description' and 'owner' for each action) > > A few functions are moved from tools/metrics/histograms to tools/metrics/common to be shared by tools/metrics and tools/histograms. > > BUG=340735 > NOTRY=true > > Review URL: https://codereview.chromium.org/149503005 TBR=yiyaoliu@chromium.org Review URL: https://codereview.chromium.org/188793003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255404 0039d316-1c4b-4281-b951-d872f2087c98 --- PRESUBMIT.py | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) (limited to 'PRESUBMIT.py') diff --git a/PRESUBMIT.py b/PRESUBMIT.py index c7b9fde..1e6526f 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -1014,32 +1014,28 @@ def _CheckCygwinShell(input_api, output_api): def _CheckUserActionUpdate(input_api, output_api): """Checks if any new user action has been added.""" - if any('actions.xml' == input_api.os_path.basename(f) for f in + if any('chromeactions.txt' == input_api.os_path.basename(f) for f in input_api.LocalPaths()): - # If actions.xml is already included in the changelist, the PRESUBMIT - # for actions.xml will do a more complete presubmit check. + # If chromeactions.txt is already included in the changelist, the PRESUBMIT + # for chromeactions.txt will do a more complete presubmit check. return [] + with open('tools/metrics/actions/chromeactions.txt') as f: + current_actions = f.read() + file_filter = lambda f: f.LocalPath().endswith(('.cc', '.mm')) action_re = r'[^a-zA-Z]UserMetricsAction\("([^"]*)' - current_actions = None for f in input_api.AffectedFiles(file_filter=file_filter): for line_num, line in f.ChangedContents(): match = input_api.re.search(action_re, line) if match: - # Loads contents in tools/metrics/actions/actions.xml to memory. It's - # loaded only once. - if not current_actions: - with open('tools/metrics/actions/actions.xml') as actions_f: - current_actions = actions_f.read() - # Search for the matched user action name in |current_actions|. for action_name in match.groups(): - action = 'name="{0}"'.format(action_name) - if action not in current_actions: + name_pattern = r'\t%s\n' % action_name + if name_pattern not in current_actions: return [output_api.PresubmitPromptWarning( 'File %s line %d: %s is missing in ' - 'tools/metrics/actions/actions.xml. Please run ' - 'tools/metrics/actions/extract_actions.py to update.' + 'tools/metrics/actions/chromeactions.txt. Please run ' + 'tools/metrics/actions/extract_actions.py --hash to update.' % (f.LocalPath(), line_num, action_name))] return [] -- cgit v1.1