summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPavlo Shchelokovskyy <shchelokovskyy@gmail.com>2018-12-14 13:16:48 +0200
committerPavlo Shchelokovskyy <shchelokovskyy@gmail.com>2018-12-14 13:20:12 +0200
commitdd5e0302075322a8eb44944098ff01643bcc7797 (patch)
treee8c48a1a2e156b8032918d4f285b304e80336dc3
parenteb3187863bfc3f60a5ea0f136ec3c14a8c8d6d71 (diff)
Restrict bashate to devstack/lib instead of lib
`lib` is too general in *nix systems and CI can clone the project to directories that contain it (like `/var/lib/`) which leads to bashate running on files it should not run (like `devstack/settings`) and fail. Change-Id: If4ded8a15ecda5abf9a92f15bdbc8204c025df74
Notes
Notes (review): Code-Review+2: Mark Goddard <mark@stackhpc.com> Code-Review+2: Vladyslav Drok <vdrok@mirantis.com> Workflow+1: Vladyslav Drok <vdrok@mirantis.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 19 Dec 2018 17:12:14 +0000 Reviewed-on: https://review.openstack.org/625222 Project: openstack/networking-generic-switch Branch: refs/heads/master
-rwxr-xr-xtools/run_bashate.sh2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/run_bashate.sh b/tools/run_bashate.sh
index c3993a9..eef129a 100755
--- a/tools/run_bashate.sh
+++ b/tools/run_bashate.sh
@@ -21,7 +21,7 @@ find "$@" -not \( -type d -name .?\* -prune \) \
21 -not -name \*.py \ 21 -not -name \*.py \
22 \( \ 22 \( \
23 -name \*.sh -or \ 23 -name \*.sh -or \
24 -wholename \*/lib/\* -or \ 24 -wholename \*/devstack/lib/\* -or \
25 -wholename \*/tools/\* \ 25 -wholename \*/tools/\* \
26 \) \ 26 \) \
27 -print0 | xargs -0 bashate -v -iE006 -eE005,E042 27 -print0 | xargs -0 bashate -v -iE006 -eE005,E042