Bug 192023 – Remove Postprocess Headers dependencies

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Keith Rollin]

JavaScriptCore'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 change 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.

[Radar WebKit Bug Importer]

<rdar://problem/46283377>

[Keith Rollin]

Created attachment 355755 [details]
Patch

[Mark Lam]

Comment on attachment 355755 [details]
Patch

r=me if EWS bots are green.

[EWS Watchlist]

Comment on attachment 355755 [details]
Patch

Attachment 355755 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/10172966

New failing tests:
media/no-fullscreen-when-hidden.html

[EWS Watchlist]

Created attachment 355811 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6

[WebKit Commit Bot]

Comment on attachment 355755 [details]
Patch

Clearing flags on attachment: 355755

Committed r238612: <https://trac.webkit.org/changeset/238612>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.