check-code: allow "grep pattern filename-containing-dash-a"
authorMartin von Zweigbergk <martinvonz@google.com>
Tue, 02 Feb 2016 21:20:04 -0800
changeset 27989 e77ac31b64a1
parent 27985 79139c7a88bd
child 27990 96bfd2875213
check-code: allow "grep pattern filename-containing-dash-a" We're trying to forbid "grep -a" and unintentionally complained even if the "-a" was part of the filename. Requiring a space before "-a" to match is probably good enough.
contrib/check-code.py
--- a/contrib/check-code.py	Fri Feb 05 15:18:40 2016 +0100
+++ b/contrib/check-code.py	Tue Feb 02 21:20:04 2016 -0800
@@ -90,7 +90,7 @@
     (r'pushd|popd', "don't use 'pushd' or 'popd', use 'cd'"),
     (r'\W\$?\(\([^\)\n]*\)\)', "don't use (()) or $(()), use 'expr'"),
     (r'grep.*-q', "don't use 'grep -q', redirect to /dev/null"),
-    (r'(?<!hg )grep.*-a', "don't use 'grep -a', use in-line python"),
+    (r'(?<!hg )grep.* -a', "don't use 'grep -a', use in-line python"),
     (r'sed.*-i', "don't use 'sed -i', use a temporary file"),
     (r'\becho\b.*\\n', "don't use 'echo \\n', use printf"),
     (r'echo -n', "don't use 'echo -n', use printf"),