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

feat(flake.nix/inputs/nixpkgs): Switch nixos-24.11 -> staging-next-24.11 to fix curl v8.11.0 regression #187

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

PetarKirov
Copy link
Collaborator

@PetarKirov PetarKirov commented Nov 25, 2024

curl v8.11.0 was released with a bug that causes git fetch (and related commands, including Nix's fetchers) to fail due to failure to parse .netrc files. Luckily, after the bug was reported upstream, a fix was quickly proposed and merged by its main developer. Not long after, the upstream curl patch was backported to the Nixpkgs curl derivation.

Given that we rely on .netrc files this bug is a showstopper for us, so its important that we switch to the patched curl version ASAP. Instead of waiting for the build to land on the nixos-24.11 branch, we will switch directly to the staging-next-24.11 branch, which already includes the fix.

…ng-next-24.11`

`curl` v8.11.0 was released with a bug that causes `git fetch` (and many
similar commands, including Nix's [fetchers][3]) to fail due to failure to parse
`.netrc` files. Luckily, after the bug was [reported upstream][1], a fix was quickly
proposed and merged by its main [developer][2]. Not long after, the upstream
`curl` patch was [backported][5] to the Nixpkgs `curl` derivation.

Given that we rely on `.netrc` files this bug is a showstopper for us, so its
important that we switch to the patched `curl` version ASAP. Instead of [waiting
for the build to land][5] on the `nixos-24.11` branch, we will switch directly
to the `staging-next-24.11` branch, which already includes the fix.

[1]: curl/curl#15496
[2]: curl/curl@f5c6169
[3]: NixOS/nixpkgs#356114
[4]: NixOS/nixpkgs#356133
[5]: https://nixpk.gs/pr-tracker.html?pr=356660
@PetarKirov PetarKirov force-pushed the build/flake-inputs/update-2024-11-25 branch from b0e328d to b9e1251 Compare November 25, 2024 19:25
@PetarKirov PetarKirov changed the title feat(flake.nix/inputs): Temporary switch from nixos-24.11 to staging-next-24.11 feat(flake.nix/inputs/nixpkgs): Switch nixos-24.11 -> staging-next-24.11 to fix curl v8.11.0 regression Nov 25, 2024
Copy link
Contributor

Thanks for your Pull Request!

Below you will find a summary of the cachix status of each package, for each supported platform.

package x86_64-linux x86_64-darwin aarch64-darwin
cachix ❌ build failed ✅ cached ✅ cached
dcd ✅ cached 🚫 not supported 🚫 not supported
dmd ✅ cached 🚫 not supported 🚫 not supported
dscanner ✅ cached 🚫 not supported 🚫 not supported
dub ✅ cached ✅ cached ✅ cached
folder-size-metrics ✅ cached 🚫 not supported 🚫 not supported
foundry ✅ cached ✅ cached ✅ cached
geth ✅ cached 🚫 not supported ✅ cached
grafana-agent ✅ cached 🚫 not supported ✅ cached
ldc ✅ cached 🚫 not supported 🚫 not supported
lido-withdrawals-automation ✅ cached ✅ cached ✅ cached
mcl ✅ cached 🚫 not supported 🚫 not supported
mev-boost ✅ cached 🚫 not supported 🚫 not supported
nethermind ✅ cached 🚫 not supported 🚫 not supported
nimbus-eth2 ✅ cached 🚫 not supported 🚫 not supported
nix ❌ build failed ❌ build failed ❌ build failed
nix-eval-jobs ✅ cached ✅ cached ❌ build failed
nix-fast-build ✅ cached ✅ cached ❌ build failed
pyroscope ✅ cached ✅ cached ✅ cached
rustToolchain ✅ cached ✅ cached ✅ cached
serve-d ✅ cached 🚫 not supported 🚫 not supported
terraform ✅ cached 🚫 not supported 🚫 not supported
terranix ✅ cached 🚫 not supported 🚫 not supported
validator-ejector ✅ cached 🚫 not supported 🚫 not supported
web3signer ✅ cached 🚫 not supported 🚫 not supported

@PetarKirov PetarKirov merged commit beb84f2 into main Nov 25, 2024
14 checks passed
@PetarKirov PetarKirov deleted the build/flake-inputs/update-2024-11-25 branch November 25, 2024 19:30
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.

1 participant