Skip to content

c-ares: fix IPv6 link-local DNS server support#462151

Merged
reckenrode merged 1 commit intoNixOS:stagingfrom
reckenrode:push-zsnqustrmvqt
Nov 18, 2025
Merged

c-ares: fix IPv6 link-local DNS server support#462151
reckenrode merged 1 commit intoNixOS:stagingfrom
reckenrode:push-zsnqustrmvqt

Conversation

@reckenrode
Copy link
Contributor

@reckenrode reckenrode commented Nov 16, 2025 •
edited by niklaskorz
Loading

My network only advertises an IPv6 link-local DNS server. After #451579, Lix is basically broken for me because it can’t access GitHub or cache.nixos.org.

Fortunately, this is fixed upstream but not yet available in a release, so the patch can be cherry-picked.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@reckenrode
Copy link
Contributor Author

reckenrode commented Nov 16, 2025

Note that I can work around this on my Mac using my DNS server’s unadvertised IPv4 address, but fixing my servers to use that requires config changes I’d rather not make. Fortunately, I haven’t actually updated them yet, so they still work.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Nov 16, 2025
niklaskorz
niklaskorz reviewed Nov 16, 2025
View reviewed changes
@nixpkgs-ci nixpkgs-ci bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Nov 16, 2025
@niklaskorz
Copy link
Member

niklaskorz commented Nov 16, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 462151 --package c-ares --package curl --package fetchurl
Commit: 93e142f083a31d4d500ffb8f5059a7936afdc9a8

x86_64-linux

✅ 9 packages built:
  • c-ares
  • c-ares.dev (c-ares.dev.dev, c-ares.dev.man)
  • c-ares.man (c-ares.man.dev, c-ares.man.man)
  • curl
  • curl.bin (curl.bin.bin, curl.bin.debug, curl.bin.dev, curl.bin.devdoc, curl.bin.man)
  • curl.debug (curl.debug.bin, curl.debug.debug, curl.debug.dev, curl.debug.devdoc, curl.debug.man)
  • curl.dev (curl.dev.bin, curl.dev.debug, curl.dev.dev, curl.dev.devdoc, curl.dev.man)
  • curl.devdoc (curl.devdoc.bin, curl.devdoc.debug, curl.devdoc.dev, curl.devdoc.devdoc, curl.devdoc.man)
  • curl.man (curl.man.bin, curl.man.debug, curl.man.dev, curl.man.devdoc, curl.man.man)

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Nov 16, 2025
@reckenrode reckenrode added the backport staging-25.11 Backport PR automatically label Nov 16, 2025
@scarlet-storm scarlet-storm mentioned this pull request Nov 17, 2025
Open
3 tasks
@reckenrode reckenrode added this pull request to the merge queue Nov 18, 2025
Merged via the queue into NixOS:staging with commit e390ced Nov 18, 2025
39 of 41 checks passed
@reckenrode reckenrode deleted the push-zsnqustrmvqt branch November 18, 2025 12:54
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Nov 18, 2025

Successfully created backport PR for staging-25.11:

@github-actions github-actions bot added the 8.has: port to stable This PR already has a backport to the stable release. label Nov 18, 2025
@nixpkgs-ci nixpkgs-ci bot mentioned this pull request Nov 18, 2025
Merged
reckenrode added a commit to reckenrode/nixos-configs that referenced this pull request Nov 27, 2025
@reckenrode
lix: work around c-ares regression
ea14e16 
c-ares has a bug where it ignores link-local DNS servers, which breaks
DNS on my network where only link-local DNS servers are advertised.

See: NixOS/nixpkgs#462151
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@niklaskorz niklaskorz niklaskorz approved these changes

Assignees

No one assigned

Labels

8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 12.approvals: 1 This PR was reviewed and approved by one person. backport staging-25.11 Backport PR automatically

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@reckenrode @niklaskorz