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

glibc: pull a bug work-around from Arch (i686-only ATM) #1446

Closed
wants to merge 1 commit into from

Conversation

vcunat
Copy link
Member

@vcunat vcunat commented Jan 2, 2014

This should work around some of our build problems,
e.g. http://hydra.nixos.org/build/7575893/nixlog/1/tail-reload

I tested the builds of jdk and xbmc were broken on my build machine, and now they work. It's a stdenv change for i686, but to me it seems worth it.

If noone protests, I plan to merge in about a day or two? (I don't see any Hydra branch suitable for pre-building this, for nixos:glibc-2.18 is stopped.)

@edolstra
Copy link
Member

edolstra commented Jan 2, 2014

Are you sure this only affects i686? The patch applies to sysdeps/x86_64...

I would just drop the conditional.

@viric
Copy link
Member

viric commented Jan 2, 2014

On Thu, Jan 02, 2014 at 01:29:57AM -0800, Eelco Dolstra wrote:

Are you sure this only affects i686? The patch applies to sysdeps/x86_64...

I would just drop the conditional.

I think they have some code shared for i686/x86_64, especially that of SIMD. :)

@vcunat
Copy link
Member Author

vcunat commented Jan 2, 2014

I haven't noticed affecting x86_64, and the Arch bug is also only filed against 32-bit versions (although they patch it unconditionally). Certainly feel free to drop the conditional -- it was just to limit a rebuild ATM, as it seemed to nave no effect elsewhere (I planned e.g. to port it to stdenv-updates without the condition).

Yes, it looks like using SIMD even on i686, based on runtime CPU detection. I wonder if the problem is even reported upstream.

vcunat added a commit that referenced this pull request Jan 2, 2014
vcunat added a commit that referenced this pull request Jan 4, 2014
This should work around some of our build problems,
e.g. http://hydra.nixos.org/build/7575893/nixlog/1/tail-reload

Cf. #1446; unconditional on stdenv-updates: c6b9205
@vcunat
Copy link
Member Author

vcunat commented Jan 4, 2014

Pushed to master with i686 only. I believe in stabilizing stdenv-updates soon where it's for all platforms (finally!).

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