Bug 236613 – [XCBuild] Add task inputs/outputs for "Make libWTF.a Symbolic Link"

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Elliott Williams]

[XCBuild] Add task inputs/outputs for "Make libWTF.a Symbolic Link"

[Elliott Williams]

Created attachment 451949 [details]
Add task inputs/outputs

[Radar WebKit Bug Importer]

<rdar://problem/88931997>

[Alexey Proskuryakov]

Comment on attachment 451949 [details]
Add task inputs/outputs

Out of curiosity, can the project be set up to find libWTF.a wherever it is, without a symlink? We have such behavior for everything else, like headers and dylibs.

[Elliott Williams]

(In reply to Alexey Proskuryakov from comment #3)
> Comment on attachment 451949 [details]
> Patch
> 
> Out of curiosity, can the project be set up to find libWTF.a wherever it is,
> without a symlink? We have such behavior for everything else, like headers
> and dylibs.

If we were to require that engineering builds always use a local libWTF.a, whereas production builds always use an SDK-provided libWTF.a, then we could determine this statically via a build setting.

I *think* that's the only valid use case, since an engineering build of JSC is likely to depend on WTF changes that haven't made it into the engineer's system SDK yet. Does that sound right?

[Alexey Proskuryakov]

I think that's correct.

But would that bring libWTF.a in line with how headers and dylibs are handled? If not, then I'm still curious about why it needs to be different.

[Elliott Williams]

(In reply to Alexey Proskuryakov from comment #5)
> I think that's correct.
Cool, I'll try it, and maybe we can delete this phase altogether.

> But would that bring libWTF.a in line with how headers and dylibs are
> handled? If not, then I'm still curious about why it needs to be different.
libWTF.a is unique in that it's passed to the linker via `-full_load`, which takes a file path and does not use the search paths that headers, frameworks, or libraries would typically use. Presumably, we do this because WebKit and WebCore use symbols from WTF that JavaScriptCore does not, so we can't allow the linker to strip them out.

[Elliott Williams]

Created attachment 451993 [details]
Remove build phase entirely

[EWS]

Committed r289823 (247281@main): <https://commits.webkit.org/247281@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 451993 [details].