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
softether module: fix dataDir option #35343
Conversation
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.
So much nicer! Thanks!
@GrahamcOfBorg build softether |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Failure on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Softether uses a |
@@ -48,6 +48,6 @@ stdenv.mkDerivation rec { | |||
homepage = https://www.softether.org/; | |||
license = licenses.gpl2; | |||
maintainers = [ maintainers.rick68 ]; | |||
platforms = platforms.linux; | |||
platforms = filter (p: p != "aarch64-linux") platforms.linux; |
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.
The compiler options restricts softether to exactly x86_64-linux
. So just put platforms = ['x86_64-linux'];
here.
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.
I was not precise when I said disabling aarch64
.
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.
@Mic92 x86 is supported. They use very rudimentary configure script:
cp src/makefiles/${OS}_${CPU}.mak Makefile
that just picks suitable Makefile.
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.
@Mic92 Thanks, aarch64 is disabled now, as I have no aarch64 machines to test. Also they don't support it officially yet: https://www.softether.org/4-docs/1-manual/5._SoftEther_VPN_Bridge_Manual/5.1_Operating_System_Requirements#5.1.2._Support_for_Linux
Motivation for this change
Closes #35339
cc @rick68
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)