summaryrefslogtreecommitdiffstats
path: root/tox.ini
diff options
context:
space:
mode:
authorguillaume.lambert <guillaume.lambert@orange.com>2021-12-10 21:17:30 +0100
committerguillaume.lambert <guillaume.lambert@orange.com>2021-12-12 00:23:51 +0100
commitd09f363e87c98d19ba2fccdf8f235d5cbfb908ee (patch)
tree80e489c14426ec693563b629aa1b1e5ac1e5b092 /tox.ini
parentab85a0e0c722b248e77bc8d02c1647b3bd9154ff (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.ini13
1 files changed, 3 insertions, 10 deletions
diff --git a/tox.ini b/tox.ini
index dee4f3accf..337b397a26 100644
--- a/tox.ini
+++ b/tox.ini
@@ -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 =