diff options
| author |   guillaume.lambert <guillaume.lambert@orange.com> | 2021-12-10 21:17:30 +0100 |
|---|---|---|
| committer | guillaume.lambert <guillaume.lambert@orange.com> | 2021-12-12 00:23:51 +0100 |
| commit | d09f363e87c98d19ba2fccdf8f235d5cbfb908ee (patch) | |
| tree | 80e489c14426ec693563b629aa1b1e5ac1e5b092 /tox.ini | |
| parent | ab85a0e0c722b248e77bc8d02c1647b3bd9154ff (diff) | |
[GLOBAL] Move checkbashisms to a dedicated script
Using a dedicated shell script rather than scripts commands directly
in tox.ini configuration is usually recommanded.
It avoids quoting hell issues and ensures a better virtualenv isolation.
Issue-ID: OOM-2643
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Co-authored-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com>
Change-Id: Ic4764f628d6f8c3bb4d0dd36a0559603c80f6c4a
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 13 |
1 files changed, 3 insertions, 10 deletions
@@ -57,17 +57,10 @@ commands = [testenv:checkbashisms] deps = -whitelist_externals = sh - find - checkbashisms +whitelist_externals = + {toxinidir}/.ci/check-bashisms.sh commands = - sh -c 'which checkbashisms>/dev/null || sudo yum install devscripts-minimal || sudo apt-get install devscripts \ - || (echo "checkbashisms command not found - please install it (e.g. sudo apt-get install devscripts | \ - yum install devscripts-minimal )" >&2 && exit 1)' - find . -not -path '*/\.*' -name *.sh -exec checkbashisms \{\} + - find . -not -path '*/\.*' -name *.failover -exec checkbashisms -f \{\} + - sh -c "! find . -not -path '*/\.*' -name *.sh -exec grep 'local .*=' \{\} + || exit 2" - sh -c "! find . -not -path '*/\.*' -name *.failover -exec grep 'local .*=' \{\} + || exit 2" + {toxinidir}/.ci/check-bashisms.sh [testenv:autopep8] deps = |