michaeleisel / zld

A faster version of Apple's linker
MIT License
1.19k stars 50 forks source link

Mac Catalyst linker issues #43

Closed steipete closed 4 years ago

steipete commented 4 years ago

zld works great for us overall. We cannot yet enable it for Mac Catalyst. It seems the linker pulls off some tricks to differentiate between AppKit and iOS-versions of the frameworks.

We see the following error:

Building for Mac Catalyst, but linking in .tbd built for , file '/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/System/Library/Frameworks//CoreImage.framework/CoreImage.tbd' for architecture x86_64

This might be fixed once https://github.com/michaeleisel/zld/issues/40 is resolved.

Thank you for zld! Gave the project a shoutout on my blog.

michaeleisel commented 4 years ago

new upgrade is in master, https://github.com/michaeleisel/zld/pull/44 , there will also be a release soon i think

michaeleisel commented 4 years ago

the new version doesn't fix it. it looks like possibly a tapi issue, will investigate further

michaeleisel commented 4 years ago

possible fix, albeit with lots of linker warnings, here: https://github.com/michaeleisel/zld/pull/50

steipete commented 4 years ago

Thanks for your work on this! I'll give this a go this week.

steipete commented 4 years ago

I've just did a quick test with zld 1.1 from brew + Xcode 12b3, same error.

michaeleisel commented 4 years ago

Are you using the new linker flag as per https://github.com/michaeleisel/zld/pull/55#issuecomment-655798318

michaeleisel commented 4 years ago

Closing, feel free to reopen if you can confirm that you're using the new flag (also mentioned in the release notes). I'll update the readme as well.

michaeleisel commented 4 years ago

Fixed properly in https://github.com/michaeleisel/zld/pull/59