[tools] winetest/cron: Fix archiving/purging old data.

Francois Gouget fgouget at codeweavers.com
Sat Apr 10 05:24:10 CDT 2021


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
The typo broke winetest.cron, preventing it from processing the reports
in the queue so that the website stayed stuck with the results from the
7th. Sorry. I should have noticed earlier :-(

Also the reason for the parentheses is so that a failure in them stops 
that part of the code but not the rest (set -e). At least I assume 
that's why they were put there in the first place.
---
 winetest/winetest.cron | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/winetest/winetest.cron b/winetest/winetest.cron
index 7c83e5d18..6806f6ef1 100755
--- a/winetest/winetest.cron
+++ b/winetest/winetest.cron
@@ -83,7 +83,7 @@ then
         tar cf - "$dir" | nice xz -c -T 0 -9 >"$dir.tar.xz"
         touch -r "$dir" "$dir.tar.xz"
         rm -rf "$dir"
-    fi )
+    ) fi
 
     # remove old build reports
     if [ -n "$build_expire" ]
@@ -92,7 +92,7 @@ then
         cd old-data
         find . -mtime "+$build_expire" -name "*.tar.*" -print0 | \
             xargs -0 rm -f
-    fi )
+    ) fi
 
     # remove old winetest binaries
     if [ -n "$winetest_expire" ]
@@ -102,7 +102,7 @@ then
         cd builds
         find . -mtime "+$winetest_expire" -name "winetest*.exe" -print0 | \
             xargs -0 rm -f
-    fi )
+    ) fi
 
     # remove old queued files
     if [ -n "$queue_expire" ]
-- 
2.20.1



More information about the wine-devel mailing list