RESOLVED FIXED 197072
Add postprocess-header-rule scripts 
https://bugs.webkit.org/show_bug.cgi?id=197072
Summary Add postprocess-header-rule scripts
Keith Rollin
Reported 2019-04-18 13:53:45 PDT
Several projects have post-processing build phases where exported headers are tweaked after they've been copied. This post-processing is performed via scripts called postprocess-headers.sh. We are now transitioning to a build process where the post-processing is performed at the same time as the exporting/copying. To support this process, add similar scripts named postprocess-header-rule, which are geared towards processing a single file at a time rather than all exported files at once. These scripts are not used at the moment; they will come into use in an imminent patch. Note that I've named these postprocess-header-rule rather than postprocess-header-rule.sh. Scripts in Tools/Scripts do not have suffixes indicating how the tool is implemented. Scripts in per-project Scripts folders appear to be mixed regarding the use of suffixes. I'm opting here to follow the Tools/Scripts convention, with the expectation that over time we completely standardize on that.
Attachments
Patch (23.25 KB, patch)
2019-04-18 13:57 PDT, Keith Rollin 		no flags
DetailsFormatted DiffDiff
Patch (27.11 KB, patch)
2019-04-18 14:17 PDT, Keith Rollin 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ews202 for win-future (13.03 MB, application/zip)
2019-04-18 16:14 PDT, EWS Watchlist 		no flags
Details
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2019-04-18 13:53:54 PDT
<rdar://problem/50027299>
Keith Rollin
Comment 2 2019-04-18 13:55:10 PDT
> We are now transitioning to a build process where the post-processing is performed at the same time as the exporting/copying. Because of XCBuild.
Keith Rollin
Comment 3 2019-04-18 13:57:35 PDT
Created attachment 367751 [details] Patch
Keith Rollin
Comment 4 2019-04-18 14:07:49 PDT
Comment on attachment 367751 [details] Patch This patch is incomplete. Taking it down.
Keith Rollin
Comment 5 2019-04-18 14:13:48 PDT
I want to add that a build rule is also being added that makes use of these scripts. This build rule is not currently active, but will be later.
Keith Rollin
Comment 6 2019-04-18 14:17:15 PDT
Created attachment 367755 [details] Patch
EWS Watchlist
Comment 7 2019-04-18 16:14:38 PDT
Comment on attachment 367755 [details] Patch Attachment 367755 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/11920186 New failing tests: js/dom/custom-constructors.html
EWS Watchlist
Comment 8 2019-04-18 16:14:50 PDT
Created attachment 367771 [details] Archive of layout-test-results from ews202 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews202 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Keith Rollin
Comment 9 2019-04-18 17:26:21 PDT
My changes here don't affect Windows, so turning off cq-.
Brent Fulgham
Comment 10 2019-04-19 13:51:24 PDT
(In reply to Keith Rollin from comment #9) > My changes here don't affect Windows, so turning off cq-. Agreed. Windows is CMake.
WebKit Commit Bot
Comment 11 2019-04-19 14:07:36 PDT
Comment on attachment 367755 [details] Patch Clearing flags on attachment: 367755 Committed r244466: <https://trac.webkit.org/changeset/244466>
WebKit Commit Bot
Comment 12 2019-04-19 14:07:38 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.