Merge lp:~dooferlad/linaro-license-protection/manifest_generation_fix into lp:~linaro-automation/linaro-license-protection/trunk

[James Tunnicliffe]

Should fix up manifest generation. The old script only generated a manifest of the files being copied during that run. Since files are copied in several stages this resulted in files not being listed.

When this is deployed and after it is proved to work, we should delete all generated MANIFEST files from snapshots.linaro.org because they are incomplete and will break the web UI file listing.

[Paul Sokolovsky (pfalcon)]

I told it would be easier to do on android-build side ;-).

Are you sure this works and as expected? We actually have 2 transfers, 1st one transfer main bunch of files, 2nd one only lava-job-info. So, I'd expect to see >>MANIFEST, or otherwise expect it to skip updating manifest if it exists, or if the file in lava-job-info (no big point to include it in artifact list imho).

review: Needs Information

[Paul Sokolovsky (pfalcon)]

Ok, the file is just regenerated from final d/l dir each time, works good too.

review: Approve

=== modified file 'scripts/jenkins-post-www.sh'
--- scripts/jenkins-post-www.sh	2012-01-27 17:16:41 +0000
+++ scripts/jenkins-post-www.sh	2012-02-01 15:54:18 +0000
@@ -24,11 +24,11 @@
 job_dir=$(dirname $build_path)
 username=`echo "$job_dir" | cut -d_ -f1`
 jobname=`echo "$job_dir" | cut -d_ -f2-`
-pushd $BASE_PATH/$build_path
-find * -xtype f > MANIFEST
-popd
+
 echo -n "Moving $BASE_PATH/$build_path to $TARGET_PATH/~$username/$jobname/... " &&
 (mkdir -p "$TARGET_PATH/~$username/$jobname" && \
      cp -a $BASE_PATH/"$build_path" "$TARGET_PATH/~$username/$jobname/" && \
      rm -rf $BASE_PATH/"$build_path" && \
      echo "done")
+
+cd "$TARGET_PATH/~$username/$jobname/" && find * -xtype f > MANIFEST