Skip to content

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