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 broken perl build due to impurity and glibc-locales build with arbitrary nix store path #1273

Closed
wants to merge 2 commits into from

Conversation

wavewave
Copy link
Contributor

Perl build has an impurity (Issue #386), this fix resolves it.

Also, if I build glibc-locales with non standard nix store location, the build is not successful due to a bug in locales.nix. This is a fix.

vcunat and others added 2 commits November 24, 2013 18:03
This probably solves problems on some platforms that expose /usr and
/lib during build, see NixOS#386.
previously, $TMPDIR/nix/store is hard-coded so that only building and installing is correct with /nix/store
@shlevy
Copy link
Member

shlevy commented Nov 25, 2013

This changes stdenv, right? If so this should be opened against the stdenv-updates branch.

vcunat pushed a commit that referenced this pull request Nov 25, 2013
previously, $TMPDIR/nix/store is hard-coded so that only building and installing is correct with /nix/store
See #1273.
@vcunat
Copy link
Member

vcunat commented Nov 25, 2013

The first patch does, and it has been in stdenv-updates for about 8 months :-) There are many good things in there and it seems to finally stabilize... I'm hopeful we can truly merge it within several weeks.

I push the second one to master, after verifying it builds and doesn't change stdenv.

@vcunat vcunat closed this Nov 25, 2013
@vcunat
Copy link
Member

vcunat commented Nov 25, 2013

I took the liberty to amend the description a little.

@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants