Merge lp:~pfalcon/linaro-license-protection/tweak-reshuffle-files into lp:~linaro-automation/linaro-license-protection/trunk

[Paul Sokolovsky]

Additional changes made to reshuffle-files handler script past initial launch. They unbroke few more builds not handled by initial version.

[Данило Шеган (danilo)]

We definitely need to start adding tests for this script as well.

I also wonder if you really want to exit when dir is not found: is lava_job_info transferred directly into the proper location instead?

review: Needs Information

[Данило Шеган (danilo)]

Clarified on IRC: no dir will be there when LAVA submission is disabled. Thanks!

review: Approve

=== modified file 'scripts/jenkins-post-www.sh'
--- scripts/jenkins-post-www.sh	2012-01-12 15:30:56 +0000
+++ scripts/jenkins-post-www.sh	2012-01-13 13:37:31 +0000
@@ -10,15 +10,15 @@
     exit 1;
 fi
 
-# Paranoid security
-if echo "$build_path" | grep -q "\."; then
-    echo "No dots in build names please"
+# Paranoid security - alarm on 2 dots separated only by 0 or more backslahes
+if echo "$build_path" | grep -q -E '\.\\*\.'; then
+    echo "No double-dots in build names please"
     exit 1
 fi
 
 if [ ! -d $BASE_PATH/$build_path ]; then
-    echo "ERROR: Expected directory $BASE_PATH/$build_path does not exist"
-    exit 1
+    echo "WARNING: Expected directory $BASE_PATH/$build_path does not exist"
+    exit 0
 fi
 
 job_dir=$(dirname $build_path)