Changeset 175436 in webkit

Timestamp:

Oct 31, 2014 4:37:25 PM (10 years ago)

Author:

mitz@apple.com

Message:

llvmForJSC build emits a linker warning that it can't find usr/local/LLVMForJavaScriptCore/lib under the build directory
​https://bugs.webkit.org/show_bug.cgi?id=138259

Reviewed by Geoff Garen.

• Scripts/copy-webkitlibraries-to-product-directory: Create this directory even if it’s

going to be empty to silence the warning.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/copy-webkitlibraries-to-product-directory (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2014-10-31  Dan Bernstein  <mitz@apple.com>
+
+        llvmForJSC build emits a linker warning that it can't find usr/local/LLVMForJavaScriptCore/lib under the build directory
+        https://bugs.webkit.org/show_bug.cgi?id=138259
+
+        Reviewed by Geoff Garen.
+
+        * Scripts/copy-webkitlibraries-to-product-directory: Create this directory even if it’s
+        going to be empty to silence the warning.
+
 2014-10-31  Adrian Perez de Castro  <aperez@igalia.com>
 

trunk/Tools/Scripts/copy-webkitlibraries-to-product-directory

@@ -161 +161 @@
 
     my $libraryDir = $useFullLibPaths ? "$productDir$llvmPrefix/lib" : $productDir;
-    (system("mkdir", "-p", $libraryDir) == 0) or die;
+    # Always create a directory at the full library path, because the JavaScriptCore build emits a warning if it's not there.
+    (system("mkdir", "-p", "$productDir$llvmPrefix/lib") == 0) or die;
 
     # Determine where to get LLVM binaries and headers.