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

Interpreter: Don't link librt and libdl on GNU systems #12038

Merged

Conversation

1player
Copy link
Contributor

@1player 1player commented Apr 29, 2022

glibc 2.34 has merged librt and libdl into libc, so they're not
available as shared libraries. The symbols are still available to the
interpreter because they are loaded in the compiler.

Fixes #12036

glibc 2.34 has merged librt and libdl into libc, so they're not
available as shared libraries. The symbols are still available to the
interpreter because they are loaded in the compiler.
@Blacksmoke16 Blacksmoke16 added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:compiler:interpreter platform:linux-gnu labels Apr 29, 2022
@yxhuvud
Copy link
Contributor

yxhuvud commented Apr 29, 2022

2.34 is less than a year old though. Would it make sense to check the glibc version? Dunno if the machinery to do that is available though.

@1player
Copy link
Contributor Author

1player commented Apr 30, 2022

I don't think there is machinery to check glibc's version, I've mostly done what @straight-shoota did previously with #11807

Copy link
Member

@beta-ziliani beta-ziliani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@beta-ziliani beta-ziliani added this to the 1.5.0 milestone May 12, 2022
@oprypin oprypin self-requested a review May 12, 2022 20:10
@oprypin oprypin removed this from the 1.5.0 milestone May 12, 2022
@oprypin oprypin removed their request for review May 13, 2022 16:34
@oprypin oprypin added this to the 1.5.0 milestone May 13, 2022
@straight-shoota straight-shoota changed the title Interpreter: Don't link with librt and libdl on GNU systems Interpreter: Don't link librt and libdl on GNU systems May 16, 2022
@straight-shoota straight-shoota merged commit 12de4e1 into crystal-lang:master May 16, 2022
@1player 1player deleted the fix-glibc-2.34-interpreter branch May 16, 2022 18:49
I3oris added a commit to I3oris/ic that referenced this pull request Jun 12, 2022
I3oris added a commit to I3oris/ic that referenced this pull request Jun 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:linux-gnu topic:compiler:interpreter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

crystal interpreter cannot find librt
6 participants