From a1fd7cea97192ff1d08d80e59d3d71e4e32608e7 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Thu, 21 Nov 2013 12:26:43 +0000 Subject: [PATCH] cr-for-branches: when doing pre-pull from incoming, take out lock correctly I.e. with -f if caller specified -q, rather than always using -w. -q would make the script carry on if it didn't manage to get the lock and do the pull, which would be wrong. -w is wrong because it can block for another long-running script. --- cr-for-branches | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cr-for-branches b/cr-for-branches index a78d34c..4c26a69 100755 --- a/cr-for-branches +++ b/cr-for-branches @@ -34,7 +34,12 @@ export LOGFILE : ${BRANCHES:=osstest xen-4.0-testing xen-4.1-testing xen-4.2-testing xen-4.3-testing xen-unstable qemu-upstream-unstable qemu-upstream-4.2-testing qemu-upstream-4.3-testing linux-3.10 linux-3.4 linux-arm-xen ${EXTRA_BRANCHES}} export BRANCHES -with-lock-ex -w data-tree-lock bash -ec ' +fetchwlem=$wlem +if [ "x$fetchwlem" = x-q ]; then + fetchwlem=-f +fi + +with-lock-ex $fetchwlem data-tree-lock bash -ec ' exec >>$LOGFILE date printf "%s\n" "$FOR_LOGFILE" -- 2.39.5