Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set that same options on the objc-static target as the objc target #211

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

madsmtm
Copy link

@madsmtm madsmtm commented Oct 7, 2021

I still can't get static linking to work, but this makes it slightly better

@davidchisnall
Copy link
Member

Thanks. I'd like to get static linking working, but it hasn't been a priority for me (and I'd really like to add static linking to CI tests to make sure it really works).

The CMake is very old (2.x-era stuff) and doesn't look very much like modern CMake. In particular, modern CMake officially supports Objective-C, so we could remove a lot of hacks. I'd be very happy to take patches to clean it up!

The CI failures on Ubuntu look new, can you take a look at them?

@madsmtm
Copy link
Author

madsmtm commented Oct 7, 2021

Can't immediately see why the build failure should be because of this PR.

Anyhow, not very experienced in CMake (primarily Rust background), but I might find the time to tinker with it - no promises though!

@davidchisnall
Copy link
Member

From the error, it look as if libunwind is being added on static builds now, but that functionality should come from something with a different name on static builds? Or maybe Ubuntu 18.04's clang just doesn't ship a static libunwind?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants