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

testserver: use glibc binary on systems with eglibc #31

Merged
merged 1 commit into from
May 3, 2017
Merged

Conversation

benesch
Copy link
Contributor

@benesch benesch commented May 3, 2017

Some versions of Ubuntu use eglibc ("embedded glibc") instead of glibc,
so they don't match the glibc regex. Attempting to run the musl
binaries on these systems hangs forever. (We should probably look into
this separately; the musl binaries should work on these systems.) For
now, though, work around the issue by grepping for the less-common
"musl" rather than "glibc."


This change is Reviewable

Some versions of Ubuntu use eglibc ("embedded glibc") instead of glibc,
so they don't match the glibc regex.  Attempting to run the musl
binaries on these systems hangs forever. (We should probably look into
this separately; the musl binaries should work on these systems.) For
now, though, work around the issue by grepping for the less-common
"musl" rather than glibc.
@benesch benesch requested review from dt and tamird May 3, 2017 19:00
@dt
Copy link
Member

dt commented May 3, 2017

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@benesch benesch merged commit d8a93c7 into master May 3, 2017
@benesch benesch deleted the eglibc branch May 3, 2017 19:10
@tamird
Copy link
Contributor

tamird commented May 3, 2017

From slack, for posterity:

Thanks for finding and fixing this! The reason I was looking for gnu is that musl is fully static and should run anywhere. Bummer that it hangs, good thing we're not publishing those binaries anywhere. We should flip the logic when the musl build actually works though. I think this is jemalloc/jemalloc#585.

Filed as #32.

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.

3 participants