Title: [238612] trunk/Source/_javascript_Core
- Revision
- 238612
- Author
- [email protected]
- Date
- 2018-11-28 08:50:40 -0800 (Wed, 28 Nov 2018)
Log Message
Remove Postprocess Headers dependencies
https://bugs.webkit.org/show_bug.cgi?id=192023
<rdar://problem/46283377>
Reviewed by Mark Lam.
_javascript_Core's Xcode Postprocess Headers build phase used to have a
dependency on a specific handful of files. In r234227, the script used
in this phase (postprocess-headers.sh) was completely rewritten to
operate on *all* files in JSC's Public and Private headers directories
instead of just this handful. This rewrite makes the previous
dependency specification insufficient, leading to incorrect
incremental builds if the right files weren't touched. Address this by
removing the dependencies completely. This will cause
postprocess-headers.sh to always be executed, even when none of its
files are touch. Running this script all the time is OK, since it has
built-in protections against unnecessarily touching files that haven't
changed.
* _javascript_Core.xcodeproj/project.pbxproj:
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (238611 => 238612)
--- trunk/Source/_javascript_Core/ChangeLog 2018-11-28 15:30:05 UTC (rev 238611)
+++ trunk/Source/_javascript_Core/ChangeLog 2018-11-28 16:50:40 UTC (rev 238612)
@@ -1,3 +1,26 @@
+2018-11-28 Keith Rollin <[email protected]>
+
+ Remove Postprocess Headers dependencies
+ https://bugs.webkit.org/show_bug.cgi?id=192023
+ <rdar://problem/46283377>
+
+ Reviewed by Mark Lam.
+
+ _javascript_Core's Xcode Postprocess Headers build phase used to have a
+ dependency on a specific handful of files. In r234227, the script used
+ in this phase (postprocess-headers.sh) was completely rewritten to
+ operate on *all* files in JSC's Public and Private headers directories
+ instead of just this handful. This rewrite makes the previous
+ dependency specification insufficient, leading to incorrect
+ incremental builds if the right files weren't touched. Address this by
+ removing the dependencies completely. This will cause
+ postprocess-headers.sh to always be executed, even when none of its
+ files are touch. Running this script all the time is OK, since it has
+ built-in protections against unnecessarily touching files that haven't
+ changed.
+
+ * _javascript_Core.xcodeproj/project.pbxproj:
+
2018-11-27 Mark Lam <[email protected]>
ENABLE_FAST_JIT_PERMISSIONS should be false for iosmac.
Modified: trunk/Source/_javascript_Core/_javascript_Core.xcodeproj/project.pbxproj (238611 => 238612)
--- trunk/Source/_javascript_Core/_javascript_Core.xcodeproj/project.pbxproj 2018-11-28 15:30:05 UTC (rev 238611)
+++ trunk/Source/_javascript_Core/_javascript_Core.xcodeproj/project.pbxproj 2018-11-28 16:50:40 UTC (rev 238612)
@@ -10124,12 +10124,6 @@
files = (
);
inputPaths = (
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/JSBase.h",
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/JSContext.h",
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/JSManagedValue.h",
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/JSValue.h",
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/JSVirtualMachine.h",
- "$(TARGET_BUILD_DIR)/$(PUBLIC_HEADERS_FOLDER_PATH)/WebKitAvailability.h",
);
name = "Postprocess Headers";
outputPaths = (
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
