Xcode prepends line comments from WTF/Compiler.h to *.sb files
authorross.kirsling@sony.com <ross.kirsling@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Apr 2018 18:14:53 +0000 (18:14 +0000)
committerross.kirsling@sony.com <ross.kirsling@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Apr 2018 18:14:53 +0000 (18:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184166

Reviewed by Brent Fulgham.

* DerivedSources.make:
Strip ;-comments from *.sb.in files before preprocessing so we can stop treating Platform.h/Compiler.h as C89.

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

Source/WebKit/ChangeLog
Source/WebKit/DerivedSources.make

index 78678f5..cf4995d 100644 (file)
@@ -1,3 +1,13 @@
+2018-04-03  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Xcode prepends line comments from WTF/Compiler.h to *.sb files
+        https://bugs.webkit.org/show_bug.cgi?id=184166
+
+        Reviewed by Brent Fulgham.
+
+        * DerivedSources.make:
+        Strip ;-comments from *.sb.in files before preprocessing so we can stop treating Platform.h/Compiler.h as C89.
+
 2018-04-03  Chris Dumez  <cdumez@apple.com>
 
         Drop MainFrame class
index da0a52d..4f564d5 100644 (file)
@@ -217,15 +217,7 @@ all : \
        @echo Generating message receiver for $*...
        @python $(WebKit2)/Scripts/generate-messages-header.py $< > $@
 
-
-# Some versions of clang incorrectly strip out // comments in c89 code.
-# Use -traditional as a workaround, but only when needed since that causes
-# other problems with later versions of clang.
-ifeq ($(shell echo '//x' | $(CC) -E -P -x c -std=c89 - | grep x),)
-TEXT_PREPROCESSOR_FLAGS=-E -P -x c -traditional -w
-else
-TEXT_PREPROCESSOR_FLAGS=-E -P -x c -std=c89 -w
-endif
+TEXT_PREPROCESSOR_FLAGS=-E -P -w
 
 ifneq ($(SDKROOT),)
        SDK_FLAGS=-isysroot $(SDKROOT)
@@ -245,7 +237,7 @@ all: $(SANDBOX_PROFILES)
 
 %.sb : %.sb.in
        @echo Pre-processing $* sandbox profile...
-       $(CC) $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(TEXT_PREPROCESSOR_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" $< > $@
+       grep -o '^[^;]*' $< | $(CC) $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(TEXT_PREPROCESSOR_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" - > $@
 
 AUTOMATION_PROTOCOL_GENERATOR_SCRIPTS = \
        $(JavaScriptCore_SCRIPTS_DIR)/cpp_generator_templates.py \