REGRESSION(183583): [Mac] make without SDKROOT has issues
authorjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 30 Apr 2015 02:39:45 +0000 (02:39 +0000)
committerjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 30 Apr 2015 02:39:45 +0000 (02:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=144431

Reviewed by Dan Bernstein.

.:

* Source/Makefile:
Assume an empty SDKROOT means an macosx variant, so only
enable settings if the SDKROOT is not empty and does not
contain "macosx".

Tools:

* Makefile:
* WebKitTestRunner/Makefile:
Assume an empty SDKROOT means an macosx variant, so only
enable settings if the SDKROOT is not empty and does not
contain "macosx".

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

ChangeLog
Source/Makefile
Tools/ChangeLog
Tools/Makefile
Tools/WebKitTestRunner/Makefile

index f5de990..90c4d6a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2015-04-29  Joseph Pecoraro  <pecoraro@apple.com>
+
+        REGRESSION(183583): [Mac] make without SDKROOT has issues
+        https://bugs.webkit.org/show_bug.cgi?id=144431
+
+        Reviewed by Dan Bernstein.
+
+        * Source/Makefile:
+        Assume an empty SDKROOT means an macosx variant, so only
+        enable settings if the SDKROOT is not empty and does not
+        contain "macosx".
+
 2015-04-29  Martin Robinson  <mrobinson@igalia.com>
 
         Fix the GTK+ build after r183584
index 2cdf3c3..99d2df6 100644 (file)
@@ -1,7 +1,9 @@
 MODULES = bmalloc WTF JavaScriptCore ThirdParty/ANGLE WebCore WebInspectorUI WebKit WebKit2
 
-ifeq (,$(findstring macosx,$(SDKROOT)))
-       MODULES = bmalloc WTF JavaScriptCore ThirdParty/ANGLE WebCore WebKit WebKit2
+ifneq (,$(SDKROOT))
+       ifeq (,$(findstring macosx,$(SDKROOT)))
+               MODULES = bmalloc WTF JavaScriptCore ThirdParty/ANGLE WebCore WebKit WebKit2
+       endif
 endif
 
 all:
index 1ac4438..949a999 100644 (file)
@@ -1,3 +1,16 @@
+2015-04-29  Joseph Pecoraro  <pecoraro@apple.com>
+
+        REGRESSION(183583): [Mac] make without SDKROOT has issues
+        https://bugs.webkit.org/show_bug.cgi?id=144431
+
+        Reviewed by Dan Bernstein.
+
+        * Makefile:
+        * WebKitTestRunner/Makefile:
+        Assume an empty SDKROOT means an macosx variant, so only
+        enable settings if the SDKROOT is not empty and does not
+        contain "macosx".
+
 2015-04-29  Brent Fulgham  <bfulgham@apple.com>
 
         Expand test infrastructure to support scrolling tests
index 163a531..dcdb4d8 100644 (file)
@@ -1,7 +1,9 @@
 MODULES = DumpRenderTree WebKitTestRunner MiniBrowser ../Source/ThirdParty/gtest/xcode TestWebKitAPI
 
-ifeq (,$(findstring macosx,$(SDKROOT)))
-       MODULES = DumpRenderTree WebKitTestRunner ../Source/ThirdParty/gtest/xcode TestWebKitAPI
+ifneq (,$(SDKROOT))
+       ifeq (,$(findstring macosx,$(SDKROOT)))
+               MODULES = DumpRenderTree WebKitTestRunner ../Source/ThirdParty/gtest/xcode TestWebKitAPI
+       endif
 endif
 
 all:
index dbcc228..34d9e5d 100644 (file)
@@ -5,9 +5,11 @@ BUILD_WEBKITTESTRUNNER = $(shell (( $(OSX_VERSION) >= 6 )) && echo "YES" )
 
 ifeq "$(BUILD_WEBKITTESTRUNNER)" "YES"
 
-ifeq (,$(findstring macosx,$(SDKROOT)))
-       ifeq (,$(findstring clean,$(MAKECMDGOALS)))
-               OTHER_OPTIONS += -target WebKitTestRunnerApp
+ifneq (,$(SDKROOT))
+       ifeq (,$(findstring macosx,$(SDKROOT)))
+               ifeq (,$(findstring clean,$(MAKECMDGOALS)))
+                       OTHER_OPTIONS += -target WebKitTestRunnerApp
+               endif
        endif
 endif