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 f5de9907afa7d18ccc03a233b938da44afb72374..90c4d6ad9fcaf418e7efebae0686db4526b46207 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 2cdf3c32868741fa6de13c84d6cd918ef95b9619..99d2df6a9d9596033aecb18b00ba8e5f79332456 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 1ac4438056f935daad02a858bfb345c2ce09ef4a..949a9999931b18c06a5cfc7ac7e079815979c528 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 163a5310fb25cf172e7dd29ffa2f01cbe1351eaf..dcdb4d8a4b03ca76bd846e1c668eb2b48da2f810 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 dbcc2284f00a44ae899167d7d12a621218169459..34d9e5db826b37a12a8e47c4f716d8d361ad01e9 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