comparison contrib/check-code.py @ 35251:91a7204631f1

contrib: ban find(1)'s -printf operator, as it is a GNU-ism

Differential Revision: https://phab.mercurial-scm.org/D1598

35250:84607c8b5af8

    (r'\w =  +\w', "only one space after = allowed"),
    (r'\bsed\b.*[^\\]\\n', "don't use 'sed ... \\n', use a \\ and a newline"),
    (r'env.*-u', "don't use 'env -u VAR', use 'unset VAR'"),
    (r'cp.* -r ', "don't use 'cp -r', use 'cp -R'"),
    (r'grep.* -[ABC]', "don't use grep's context flags"),
  ],
  # warnings
  [
    (r'^function', "don't use 'function', use old style"),
    (r'^diff.*-\w*N', "don't use 'diff -N'"),

35251:91a7204631f1

    (r'\w =  +\w', "only one space after = allowed"),
    (r'\bsed\b.*[^\\]\\n', "don't use 'sed ... \\n', use a \\ and a newline"),
    (r'env.*-u', "don't use 'env -u VAR', use 'unset VAR'"),
    (r'cp.* -r ', "don't use 'cp -r', use 'cp -R'"),
    (r'grep.* -[ABC]', "don't use grep's context flags"),
    (r'find.*-printf',
     "don't use 'find -printf', it doesn't exist on BSD find(1)"),
  ],
  # warnings
  [
    (r'^function', "don't use 'function', use old style"),
    (r'^diff.*-\w*N', "don't use 'diff -N'"),