Skip to content

Version-gate appending -syslibroot flag to lld - #7302

Merged
andrewrk merged 1 commit into
ziglang:masterfrom
kubkon:fix-6996
Dec 4, 2020
Merged

Version-gate appending -syslibroot flag to lld#7302
andrewrk merged 1 commit into
ziglang:masterfrom
kubkon:fix-6996

Conversation

@kubkon

@kubkonkubkon commented Dec 4, 2020

Copy link
Copy Markdown
Member

This commit version-gates appending -syslibroot flag to lld. This
is predicated upon the fact that for versions of macOS lower than
11, lld would fail to find and link against frameworks with this
flag specified.

Closes#6996

Co-authored-by: Andrew Kelley andrew@ziglang.org

This commit version-gates appending `-syslibroot` flag to lld. This
is predicated upon the fact that for versions of macOS lower than
11, lld would fail to find and link against frameworks with this
flag specified.
Co-authored-by: Andrew Kelley <andrew@ziglang.org>
@kubkon

Copy link
Copy Markdown
MemberAuthor

cc @mkeeter, if you could rebase onto this branch and lemme know if this fixes the linking issues for you, that'd be grand!

@kubkonkubkon added the os-macos macOS label Dec 4, 2020
@kubkonkubkon added this to the 0.7.1 milestone Dec 4, 2020
@kubkon
kubkon requested a review from andrewrkDecember 4, 2020 16:28
@andrewrk

Copy link
Copy Markdown
Member

cc @mkeeter, if you could rebase onto this branch and lemme know if this fixes the linking issues for you, that'd be grand!

I'm going to optimistically merge this & close the issue, so that there will be a binary available for testing with these changes in a couple hours. Of course please do let us know if there is still a problem.

@andrewrk
andrewrk merged commit c4853ba into ziglang:masterDec 4, 2020
@mkeeter

Copy link
Copy Markdown
Contributor

Thanks, I'll check out the auto-built binary this weekend (I haven't gone far enough down the rabbit hole to be building Zig from source on my own machine 😆)

@kubkon

Copy link
Copy Markdown
MemberAuthor

Thanks, I'll check out the auto-built binary this weekend (I haven't gone far enough down the rabbit hole to be building Zig from source on my own machine 😆)

Oh no, you’re missing all the fun!

@kubkon
kubkon deleted the fix-6996 branch December 4, 2020 20:24
@mkeeter

Copy link
Copy Markdown
Contributor

Confirmed that this now works on my machine!

Strangely, upgrading did broke a post-build step which runs install_name_tool on the resulting binary, due to complaints about signing. I don't think this is a Zig issue, though, and I worked around this by renaming the relevant dylib itself.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Linking against frameworks no longer working on macOS 10.13

3 participants

@kubkon@andrewrk@mkeeter