[packages/sysstat] - mv only if succeeded

[arekm]

commit 243b7463038749fa4ee327bfa2e0cdee52469fc6
Author: Arkadiusz Miśkiewicz <ar...@maven.pl>
Date:   Sat Jan 30 10:45:30 2016 +0100

    - mv only if succeeded

 sysstat.spec | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)
---
diff --git a/sysstat.spec b/sysstat.spec
index bbc16b3..18f2915 100644
--- a/sysstat.spec
+++ b/sysstat.spec
@@ -128,10 +128,13 @@ fi
 for log in /var/log/sa/sa[0-9]*; do
        if (LC_ALL=C %{_bindir}/sadf -C "$log" 2>&1 | grep -q "Current sysstat 
version cannot read the format of this file"); then
                echo "Converting file $log to current format: "
-               %{_bindir}/sadf -c "$log" > "$log.migrate"
-               chown --reference "$log" "$log.migrate"
-               chmod --reference "$log" "$log.migrate"
-               mv "$log.migrate" "$log"
+               if (%{_bindir}/sadf -c "$log" > "$log.migrate"); then
+                       chown --reference "$log" "$log.migrate"
+                       chmod --reference "$log" "$log.migrate"
+                       mv "$log.migrate" "$log"
+               else
+                       echo "$log MIGRATION FAILED." >&2
+               fi
        fi
 done
 
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/sysstat.git/commitdiff/243b7463038749fa4ee327bfa2e0cdee52469fc6

_______________________________________________
pld-cvs-commit mailing list
pld-cvs-commit@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-cvs-commit