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

Curl bug makes nix all mental. One-line fix. #28

Merged
merged 1 commit into from
Jul 2, 2012
Merged

Conversation

gitfoxi
Copy link

@gitfoxi gitfoxi commented Jun 30, 2012

Solves a hideous bug which affects lots of things including nix-env.
The question now is, how to rebuild everything that depends directly or indirectly on curl?

More about the underlying bug and its resolution here:
http://comments.gmane.org/gmane.comp.web.curl.library/33285

Bug looks like this:

curl google.com
curl: (7) Failed to connect to 74.125.224.35: Invalid argument

Solves a hideous bug which affects lots of things including nix-env.
The question now is, how to rebuild everything that depends directly or indirectly on curl?

More about the underlying bug and its resolution here:
    http://comments.gmane.org/gmane.comp.web.curl.library/33285

Bug looks like this:

    curl google.com
    curl: (7) Failed to connect to 74.125.224.35: Invalid argument
@shlevy
Copy link
Member

shlevy commented Jul 2, 2012

What system are you running into this problem on? We can update curl in stdenv-updates (the branch for changes that will cause a rebuild of pretty much every package), but unless you absolutely can't use nix on your system it will be some time until stdenv-updates is merged.

@edolstra: If this can only be worked-around by @gitfoxi by building his own stdenv, would that be enough justification for a quick bugfix stdenv-update?

@edolstra
Copy link
Member

edolstra commented Jul 2, 2012

Actually updating curl is not a problem because fetchurl is a "fixed-output" derivation that doesn't cause a rebuild if the dependencies of fetchurl change.

@shlevy
Copy link
Member

shlevy commented Jul 2, 2012

Ah, for some reason I assumed curl was part of the stdenv standard $PATH. Makes sense that it wouldn't be. I'll test the patch then merge if all is well.

shlevy added a commit that referenced this pull request Jul 2, 2012
Curl bug makes nix all mental. One-line fix.
@shlevy shlevy merged commit d068dbb into NixOS:master Jul 2, 2012
@gitfoxi
Copy link
Author

gitfoxi commented Jul 2, 2012

I'm okay because I've got the update. :)
The packages that really must be rebuilt are Nix and Perl.

I am running on a bit of a strange system. I'm trying to run /nix
along side RHEL3 from 2002 with 2.4 kernel. It's going okay, but I
think I need to downgrade glibc back to 2.3.6 or move to uclibc before
I'll ever get a web browser running because of some big change to
kernel threading technology. It might be a factor in the curl problem.
Apparently kernel threading is sketchy for several years of 2.6 as
well which sticks it to embedded developers who are stuck with this or
that antiquated kernel patched for their specific chip.

Anyway, it was easy to find people complaining about this buggy
version of curl on the internet so it might affect other people. It is
kind of insidious because it is intermittent. Since Nix has failover
logic it makes it do strange things.

On Sun, Jul 1, 2012 at 9:17 PM, Shea Levy
reply@reply.github.com
wrote:

Ah, for some reason I assumed curl was part of the stdenv standard $PATH. Makes sense that it wouldn't be. I'll test the patch then merge if all is well.


Reply to this email directly or view it on GitHub:
#28 (comment)

Michael

@shlevy
Copy link
Member

shlevy commented Jul 2, 2012

The change was already merged :) Thanks for the contribution

pikajude referenced this pull request in pikajude/nixpkgs Jan 22, 2015
…ncies, and removing unnecessary references.

Fixes #15, fixes #28, fixes #31, fixes NixOS#35.
mguentner pushed a commit to mguentner/nixpkgs that referenced this pull request Aug 20, 2017
Profpatsch pushed a commit that referenced this pull request Jun 20, 2020
hardware-configuration.nix is still relied upon for hard drive
configuration, and it handles variations of hardware (custom
configurations, subrevisions).
ethancedwards8 pushed a commit to ethancedwards8/nixpkgs that referenced this pull request Apr 2, 2021
Default outputs should include "man"
qjcg added a commit to qjcg/nixpkgs that referenced this pull request Jul 23, 2021
nat-418 added a commit to nat-418/nixpkgs that referenced this pull request Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants