Summary: | Don't migrate WebKit DOM headers in MigrateHeaders.make | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||||
Component: | WebKit Misc. | Assignee: | Alexey Proskuryakov <ap> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aakash_jain, andersca, ap, commit-queue, darin, mitz | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Local Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=160654 | ||||||||||
Attachments: |
|
Description
Aakash Jain
2017-01-30 15:08:58 PST
Created attachment 300152 [details]
Proposed patch
The current version of this patch this breaks the build. More changes are needed here, taking this. Created attachment 300863 [details]
proposed patch
Attachment 300863 [details] did not pass style-queue:
ERROR: Source/WebKit/mac/DOM/DOMTokenList.mm:28: Alphabetical sorting problem. [build/include_order] [4]
ERROR: Source/WebKit/mac/DOM/DOMTokenList.mm:32: Bad include order. Mixing system and custom headers. [build/include_order] [4]
ERROR: Source/WebKit/mac/DOM/DOMImplementation.mm:27: Alphabetical sorting problem. [build/include_order] [4]
Total errors found: 3 in 19 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Forgot to make deleting iOS headers conditional. Will fix. Comment on attachment 300863 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=300863&action=review > Source/WebKit/mac/ChangeLog:45 > + * postprocess-headers.sh: There are two SPI headers that are only exposed on iOS. > + This is the cleanest way I found to avoid exposing them on Mac (we don't migrate > + them, so it would be wrong to keep the logic in MigrateHeaders.make). Have you considered naming the two headers in the EXCLUDED_SOURCE_FILES build setting when targeting macOS? If it works, then it’s somewhat cleaner in that the files do not get copied, then deleted, in each incremental build of the framework. Created attachment 300882 [details]
proposed patch
It works, thank you Dan!
Attachment 300882 [details] did not pass style-queue:
ERROR: Source/WebKit/mac/DOM/DOMTokenList.mm:28: Alphabetical sorting problem. [build/include_order] [4]
ERROR: Source/WebKit/mac/DOM/DOMTokenList.mm:32: Bad include order. Mixing system and custom headers. [build/include_order] [4]
ERROR: Source/WebKit/mac/DOM/DOMImplementation.mm:27: Alphabetical sorting problem. [build/include_order] [4]
Total errors found: 3 in 19 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Committed http://trac.webkit.org/r211931 |