[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdesdk/scripts
From:       Michael Pyne <michael.pyne () kdemail ! net>
Date:       2005-07-31 20:26:46
Message-ID: 1122841606.758492.22719.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 441745 by mpyne:

Clarify build system cleaned message, and only display when we've actually run
rm -rf.


 M  +17 -15    kdesvn-build  


--- trunk/KDE/kdesdk/scripts/kdesvn-build #441744:441745
@@ -3147,18 +3147,20 @@
         return 1;
     }
 
-    if (-e "$builddir" &&
-        safe_system ('rm', '-rf', "$builddir"))
+    if (-e "$builddir")
     {
-        # Remove build directory for normal module.
-        error "\tUnable to clean r[$builddir].";
-        return 0; # False for this function.
+        if(safe_system ('rm', '-rf', "$builddir"))
+        {
+            # Remove build directory for normal module.
+            error "\tUnable to clean r[$builddir].";
+            return 0; # False for this function.
+        }
+
+        # Let users know we're done so they don't wonder why rm -rf is taking so
+        # long and oh yeah, why'd my HD so active?...
+        info "\tOld build system cleaned, starting new build system.";
     }
 
-    # Let users know we're done so they don't wonder why rm -rf is taking so
-    # long and oh yeah, why'd my HD so active?...
-    info "\tOld build system cleaned, starting anew.";
-
     # Now create the directory
     if (not super_mkdir ("$builddir"))
     {
@@ -3694,12 +3696,6 @@
             next;
         }
 
-        if (pretending)
-        {
-            pretend "\tWould have installed g[$module]";
-            next;
-        }
-
         # Just in case, I guess.
         update_module_environment ($module);
 
@@ -3719,6 +3715,12 @@
             }
         }
 
+        if (pretending)
+        {
+            pretend "\tWould have installed g[$module]";
+            next;
+        }
+
         next if $result != 0; # Don't delete anything if the build failed.
 
         my $remove_setting = get_option($module, 'remove-after-install');
[prev in list] [next in list] [prev in thread] [next in thread]