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

Fix debian buster linker problem #6

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

5balls
Copy link

@5balls 5balls commented Aug 7, 2021

It seems debian busters clang requires this change to the CMakeLists.txt file to link correctly.

I have set clang via update-alternatives and changed this file, then it worked.

I also installed clang-11 backport to buster but if I interpret cmake output correctly it used the normal system version of clang 7.0.1..

All of this may be obsolete with the next debian stable version appearing in about a week I think, so it may be reasonable to ignore this pull request if the new debian release fixes this and this pull request may cause other problems down the line (though I don't see why it should).

@5balls 5balls marked this pull request as draft August 10, 2021 18:33
@5balls
Copy link
Author

5balls commented Aug 10, 2021

Apparently while fixing my problem this breaks other stuff - should probably not be merged.

@MichielDerhaeg
Copy link
Contributor

Sadly, how to link against the right library for std::filesystem support depends on the OS, which version of the OS, which compiler is used, etc..
Not sure if we can get it right for every case.

This is also just a dependency of the unit test. For using the library it shouldn't be needed.

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

Successfully merging this pull request may close these issues.

2 participants