make clean does not work in OpenSource, fails in WebKitLibraries
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Aug 2014 19:20:19 +0000 (19:20 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Aug 2014 19:20:19 +0000 (19:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=135734

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2014-08-08
Reviewed by David Kilzer.

Tools:

* Scripts/copy-webkitlibraries-to-product-directory:
Add a --clean switch that removes possible WebKitSystemInterface
and LLVM libraries and headers that may have been copied to the
product directory.

WebKitLibraries:

* Makefile:
Add a "clean" target that runs copy-webkitlibraries-to-product-dir
with the new --clean switch.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@172349 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/copy-webkitlibraries-to-product-directory
WebKitLibraries/ChangeLog
WebKitLibraries/Makefile

index 6308a66..9155fde 100644 (file)
@@ -1,5 +1,17 @@
 2014-08-08  Joseph Pecoraro  <pecoraro@apple.com>
 
+        make clean does not work in OpenSource, fails in WebKitLibraries
+        https://bugs.webkit.org/show_bug.cgi?id=135734
+
+        Reviewed by David Kilzer.
+
+        * Scripts/copy-webkitlibraries-to-product-directory:
+        Add a --clean switch that removes possible WebKitSystemInterface
+        and LLVM libraries and headers that may have been copied to the
+        product directory.
+
+2014-08-08  Joseph Pecoraro  <pecoraro@apple.com>
+
         commit-log-message has extra blank line at the top of pre-populated message
         https://bugs.webkit.org/show_bug.cgi?id=135744
 
index 7fbd81e..c860192 100755 (executable)
@@ -33,6 +33,7 @@ use webkitdirs;
 my $showHelp = 0;
 my $llvm = 0;
 my $wksi = 0;
+my $clean = 0;
 my $llvmIncludePackage = "";
 my $llvmLibraryPackage = "";
 my $useFullLibPaths = 0;
@@ -49,6 +50,7 @@ Usage: $programName [options]
   --help                        Show this help message
   --[no-]llvm                   Toggle copying LLVM drops (default: $llvm)
   --[no-]wksi                   Toggle copying WebKitSystemInterface drops (default: $wksi)
+  --clean                       Clean the libraries (default: $clean)
   --use-llvm-includes=<path>    Get the LLVM inludes package from <path>
   --use-llvm-libraries=<path>   Get the LLVM libraries package from <path>
   --[no-]use-full-lib-paths     Toggle using full library paths
@@ -64,6 +66,7 @@ GetOptions(
     'help' => \$showHelp,
     'llvm!' => \$llvm,
     'wksi!' => \$wksi,
+    'clean' => \$clean,
     'use-llvm-includes=s' => \$llvmIncludePackage,
     'use-llvm-libraries=s' => \$llvmLibraryPackage,
     'use-full-lib-paths!' => \$useFullLibPaths,
@@ -302,3 +305,12 @@ if ($llvm) {
     }
 }
 
+if ($clean) {
+    print "Cleaning.\n";    
+    (system("rm", "-rf", "$productDir/usr/local/include/WebKitSystemInterface.h") == 0) or die;
+    (system("rm", "-rf", "$productDir$llvmPrefix") == 0) or die;
+    unlink glob "$productDir/libWebKitSystemInterface*" or die if glob "$productDir/libWebKitSystemInterface*";
+    unlink glob "$productDir/usr/local/lib/libWebKitSystemInterface*" or die if glob "$productDir/usr/local/lib/libWebKitSystemInterface*";
+    unlink glob "$productDir/libLLVM*" or die if glob "$productDir/libLLVM*";
+    unlink glob "$productDir/libLTO*" or die if glob "$productDir/libLTO*";
+}
index 139f0b3..1dffc26 100644 (file)
@@ -1,3 +1,14 @@
+2014-08-08  Joseph Pecoraro  <pecoraro@apple.com>
+
+        make clean does not work in OpenSource, fails in WebKitLibraries
+        https://bugs.webkit.org/show_bug.cgi?id=135734
+
+        Reviewed by David Kilzer.
+
+        * Makefile:
+        Add a "clean" target that runs copy-webkitlibraries-to-product-dir
+        with the new --clean switch.
+
 2014-08-04  Benjamin Poulain  <benjamin@webkit.org>
 
         Add a flag for the CSS Selectors level 4 implementation
index ed0ffba..c9f3535 100644 (file)
@@ -3,9 +3,13 @@ SCRIPTS_PATH = ../Tools/Scripts
 ifeq ($(shell sw_vers -productVersion | cut -d. -f-2), 10.10)
 libs:
        @true
+cleanlibs:
+       @true
 else
 libs:
        perl $(SCRIPTS_PATH)/copy-webkitlibraries-to-product-directory --wksi --llvm
+cleanlibs:
+       perl $(SCRIPTS_PATH)/copy-webkitlibraries-to-product-directory --clean
 endif
 
 all: libs
@@ -20,4 +24,7 @@ release r:
 
 analyze:
        $(SCRIPTS_PATH)/set-webkit-configuration --release
-       @$(MAKE) libs
\ No newline at end of file
+       @$(MAKE) libs
+
+clean:
+       @$(MAKE) cleanlibs