ASSIGNED Bug 223250
adjust-angle-include-paths.py is making the maintenance more difficult than it should be
https://bugs.webkit.org/show_bug.cgi?id=223250
Summary adjust-angle-include-paths.py is making the maintenance more difficult than i...
Kimmo Kinnunen
Reported 2021-03-16 05:29:51 PDT
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
Kimmo Kinnunen
Comment 1 2021-03-19 01:10:06 PDT
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
Comment 2 2021-03-19 01:14:13 PDT
The explicit headers part is addressed by bug 223470
Alexey Proskuryakov
Comment 3 2021-03-19 10:57:59 PDT
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
Comment 4 2021-03-23 05:30:16 PDT
Note You need to log in before you can comment on or make changes to this bug.