-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
rsync build varies according to kernel cmdline ipv6.disable #360152
Comments
I have two machines, one reproduces cache.nixos.org, the other does not. It looks like cache.nixos.org has Seems slightly odd to me that the rsync build would check if the local machine has ipv6 enabled.
|
Upstream issue: RsyncProject/rsync#675 |
Confirmed that re-enabling ipv6 on the original machine has now caused it to match the cache.nixos.org build. |
Per @raboof's suggestion on matrix, |
Rsync does a runtime check for ipv6 availability unless explicitly enabled/disabled, leading to a diverged build between machines if one machine has `ipv6.disable=1` kernel commandline on linux. Skip the check by always enabling it. Ref: RsyncProject/rsync#675 Fix: NixOS#360152 Signed-off-by: Peter Waller <[email protected]>
curl has the same check |
Rsync does a runtime check for ipv6 availability unless explicitly enabled/disabled, leading to a diverged build between machines if one machine has `ipv6.disable=1` kernel commandline on linux. Skip the check by always enabling it. Ref: RsyncProject/rsync#675 Fix: NixOS#360152 Signed-off-by: Peter Waller <[email protected]>
Hmm. I'm seeing the build id vary in curl so I can't confirm whether the same fix is an improvement there; I'm not seeing the text section vary though, suggesting that at least for now curl is invariant of ipv6.disable. |
Rsync's configure phase does a runtime check for ipv6 availability unless explicitly enabled/disabled, leading to a diverged build between machines if one machine has `ipv6.disable=1` kernel commandline on linux. Skip the check by always enabling it. Ref: RsyncProject/rsync#675 Fix: NixOS#360152 Signed-off-by: Peter Waller <[email protected]>
Interestingly I don't see that here. |
Fix for rsync merged in #360445. I haven't time to dig into the curl situation at the moment, if there is one. |
Issue description
Rsync is not currently reproducible. Upstream issue: RsyncProject/rsync#675
On nixos-unstable:
(or
nix-build --check -A rsync
from that commit)Reports:
Diffoscope output for the two paths show only the binaries differ, and they differ in various virtual addresses: rsync-diffoscope.txt.gz
Steps to reproduce
Technical details
The text was updated successfully, but these errors were encountered: