-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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
libreswan: Unbreak the package #101543
libreswan: Unbreak the package #101543
Conversation
On current unstable, libreswan compiles but then doesn't run because of an ABI change in NSS. There is an easy fix which is to land in the next version of libreswan, see libreswan/libreswan#334; in the meantime, we can copy this fix so that libreswan works again on unstable.
While we are at it: I tried building libreswan on darwin and got the following error:
We should probably remove the darwin platform: - platforms = platforms.linux ++ platforms.darwin ++ platforms.freebsd;
+ platforms = platforms.linux ++ platforms.freebsd; Edit: I found iproute2mac but I can't test right now if it works or not. |
Good catch, thank you @SuperSandro2000! Do you have a way to test this later? How come that libreswan ever worked on darwin? (I don't run darwin and also don't really have any clue about libreswan.) |
I can't really fiddle with network right now. Sorry.
This is like this since 2016 https://github.com/nixos/nixpkgs/blob/69d8cb4a6bf85b2a6546a20a9e8b9c9c0f2943ec/pkgs/tools/networking/libreswan/default.nix and probably no one ever tested it on darwin. I think it is save to remove darwin platform. @r-burns Can maybe you help out? |
Okay, then I'll push a commit to remove darwin for now. Thank you for your quick response. :-) If r-burns or someone else has time to look into darwin, we can always reactivate it. Right not it is broken on darwin either way, so it's not a regression to remove darwin from the supported platforms. |
See discussion at NixOS#101543. Currently, this build depends on iproute2 which is not available on darwin. This might be fixed in the future by employing iproute2mac.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the ping, yeah I think removing darwin from platforms is the right call. There are a ton of linux-only dependencies beyond iproute2, and the upstream project only claims to support linux.
Builds for me on linux. Is there a command I can run to verify the fix is working?
Thank you for the quick review!
Unfortunately, I know of no quick command for testing. You need to enable the libreswan service with some dummy configuration.
|
Result of 1 package marked as broken and skipped:
1 package built:
|
Can this pull request be merged? For as long as libreswan 3.33 is not released, this issue is unfortunately still relevant. |
Awesome, thank you @SuperSandro2000! |
On current unstable, libreswan compiles but then doesn't run because of an ABI
change in NSS.
There is an easy fix which is to land in the next version of libreswan, see
libreswan/libreswan#334; in the meantime, we can copy
this fix so that libreswan works again on unstable.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)