# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 1567837590 25200
# Node ID 4257c33e24b782936783a713d711daff4a348be6
# Parent  5ca351ba24788d7a74b9b2b4d35589ce87d48942
check-code: allow command substitution with $(command)

Both `command` and $(command) are specified by POSIX. The latter nests
better. I don't see why we shouldn't allow both (or only the latter).

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

diff -r 5ca351ba2478 -r 4257c33e24b7 contrib/check-code.py
--- a/contrib/check-code.py	Fri Jun 14 16:26:11 2019 +0100
+++ b/contrib/check-code.py	Fri Sep 06 23:26:30 2019 -0700
@@ -116,7 +116,6 @@
     (r'\bls\b.*-\w*R', "don't use 'ls -R', use 'find'"),
     (r'printf.*[^\\]\\([1-9]|0\d)', r"don't use 'printf \NNN', use Python"),
     (r'printf.*[^\\]\\x', "don't use printf \\x, use Python"),
-    (r'\$\(.*\)', "don't use $(expr), use `expr`"),
     (r'rm -rf \*', "don't use naked rm -rf, target a directory"),
     (r'\[[^\]]+==', '[ foo == bar ] is a bashism, use [ foo = bar ] instead'),
     (r'(^|\|\s*)grep (-\w\s+)*[^|]*[(|]\w',