Bug 223250
Summary: | adjust-angle-include-paths.py is making the maintenance more difficult than it should be | ||
---|---|---|---|
Product: | WebKit | Reporter: | Kimmo Kinnunen <kkinnunen> |
Component: | WebGL | Assignee: | Kimmo Kinnunen <kkinnunen> |
Status: | ASSIGNED | ||
Severity: | Normal | CC: | ap, dino, kbr, kkinnunen, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | Other | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 223470 | ||
Bug Blocks: | 221664 |
Kimmo Kinnunen
adjust-angle-include-paths.py is making the maintenance more difficult than it should be
Problems:
* The WebCore part has to jump multiple hoops just to include angle headers
* Angle headers are not complete, preventing explicit context API usage
Currently:
1) copy partial amount of public headers
2) regex-patch the headers to fix the shortcoming that the folder structure was flattened by 1
Most likely the problem should be solved one of
a) publish the public headers via publishing the folder reference of ANGLE/include
b) copy public headers via script
https://stackoverflow.com/questions/3807241/keeping-directory-structure-when-creating-frameworks-in-xcode
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Kimmo Kinnunen
In this bug we could address the difference, if it's desired:
WTF has "Copy WTF Headers" build target where it rsyncs the headers, preserving the directory structure, to the header path.
wtf does not have Xcode public/private headers, only project headers.
libwebrtc has "Copy libwebrtc headers" build target where it rsyncs the headers, preserving the directory structure, to the header path.
libwebrtc does not have Xcode public/private headers, only project headers.
ANGLE has "Adjust ANGLE paths" *build phase* where it regex-replaces some #includes to not have directories.
ANGLE has Xcode public headers.
Kimmo Kinnunen
The explicit headers part is addressed by bug 223470
Alexey Proskuryakov
At some point, we should be switching from script phases to using a copy headers phase with APPLY_RULES_IN_COPY_HEADERS, as that enables a much easier way to track dependencies. This feature is new build system only, so it cannot be done right now.
Radar WebKit Bug Importer
<rdar://problem/75734268>