summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTyler Baker <tyler.baker@linaro.org>2013-11-13 14:48:56 -0800
committerTyler Baker <tyler.baker@linaro.org>2013-11-13 14:48:56 -0800
commit5a7f707ed26afdb199f09aeab347ab5a84c03143 (patch)
tree229a5b6a41a6ade3784f4a40c0a770bd8375ec8f
parentc723df406f342f70384126d1f94bd05d269952d1 (diff)
downloadlava-pep8-5a7f707ed26afdb199f09aeab347ab5a84c03143.tar.gz
Take ignore options from the command line
Signed-off-by: Tyler Baker <tyler.baker@linaro.org>
-rwxr-xr-xrun.sh10
1 files changed, 5 insertions, 5 deletions
diff --git a/run.sh b/run.sh
index 37940f8..00ef983 100755
--- a/run.sh
+++ b/run.sh
@@ -1,5 +1,5 @@
#!/bin/bash
-# Usage ./run.sh
+# Usage ./run.sh <pep8 ignore options>
# $WORKSPACE - Jenkins Workspace Variable
# $BUILD_URL - Jenkins Build URL
@@ -45,13 +45,13 @@ function pep8_check ()
if [[ -s $WORKSPACE/pep8 ]]
then
- echo "* Test Failed: PEP8 (pep8 --ignore E501 .)" >> $WORKSPACE/results
+ echo "* Test Failed: PEP8 (pep8 --ignore $1 .)" >> $WORKSPACE/results
sed 's/^/*/g' $WORKSPACE/pep8 >> $WORKSPACE/results
echo "PEP8 CHECK FAILED:"
- pep8 --ignore E501 --show-source --show-pep8 .
+ pep8 --ignore $1 --show-source --show-pep8 .
echo "* Please see the console output from the build url below for verbose PEP8 output" >> $WORKSPACE/results
else
- echo "* Test Passed: PEP8 (pep8 --ignore E501 .)" >> $WORKSPACE/results
+ echo "* Test Passed: PEP8 (pep8 --ignore $1 .)" >> $WORKSPACE/results
fi
}
@@ -78,5 +78,5 @@ function publish_results ()
notify_committer
check_step checkout_and_rebase
-check_step pep8_check
+check_step pep8_check $1
publish_results