Fix FS #9280 (bsd tar errors out when called with -u and the destination file doesn't exist) by using -c. This requires buildzip.pl to only call the archiver once, with the additional beneft that it's more efficient (saves a call, and allows 7zip archives to become smaller).

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@18350 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 12 insertions, 16 deletions

diff --git a/tools/buildzip.pl b/tools/buildzip.pl
index 88fca97..02f2693 100755
--- a/tools/buildzip.pl
+++ b/tools/buildzip.pl
@@ -433,26 +433,22 @@ sub runone {
     # build a full install .rockbox directory
     buildzip($target, $fonts);
 
-    # create a zip file from the .rockbox dfir
-
     unlink($output);
-    if($verbose) {
-      print "$ziptool $output .rockbox >/dev/null\n";
+
+    if($fonts == 1) {
+        # Don't include image file in fonts-only package
+        undef $target;
+    }
+    if($target && ($target !~ /(mod|ajz|wma)\z/i)) {
+        # On some targets, the image goes into .rockbox.
+        copy("$target", ".rockbox/$target");
+        undef $target;
     }
-    system("$ziptool $output .rockbox >/dev/null");
 
-    if($target && ($fonts != 1)) {
-        # On some targets, rockbox.* is inside .rockbox
-        if($target !~ /(mod|ajz|wma)\z/i) {
-            copy("$target", ".rockbox/$target");
-            $target = ".rockbox/".$target;
-        }
-        
-        if($verbose) {
-            print "$ziptool $output $target >/dev/null\n";
-        }
-        system("$ziptool $output $target >/dev/null");
+    if($verbose) {
+        print "$ziptool $output .rockbox $target >/dev/null\n";
     }
+    system("$ziptool $output .rockbox $target >/dev/null");
 
     # remove the .rockbox afterwards
     rmtree('.rockbox');