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
shadowsocks-libev: supports darwin #41421
shadowsocks-libev: supports darwin #41421
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.
The build works fine, but install_name of the libraries isn't correct. If we don't fix that any usages will still be broken.
e189bd3
to
2fb3c6c
Compare
@LnL7 thanks for reviewing, fixed |
@GrahamcOfBorg build shadowsocks-libev |
Success on x86_64-darwin (full log) Attempted: shadowsocks-libev Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: shadowsocks-libev Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: shadowsocks-libev Partial log (click to expand)
|
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'm not sure why this isn't set correctly by the build in the first place. It uses cmake and that should do this automatically.
install_name_tool -change libbloom.dylib $out/lib/libbloom.dylib $out/lib/libipset.dylib | ||
|
||
for exe in $out/bin/*; do | ||
install_name_tool -change libmbedtls.dylib ${mbedtls}/lib/libmbedtls.dylib $exe |
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.
This only fixes the current library references, not the install_name (see. otool -D
and install_name_tools -id
). This means the binaries now work but anything that links against the libraries will run into the same issues.
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.
should I revert?
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.
It's ok, the package itself will work properly now and there's currently nothing in nixpkgs that links against it.
* master: (26 commits) python.pkgs.cairocffi: update tests for Cairo 1.15.12 python.pkgs.cairocffi: 0.8.0 -> 0.8.1 freecell-solver: fix build perlPackages.CPANPLUS: restore build inputs removed in #41394 Fix GHCJS 8.4/8.2 in sandboxed builds (#41411) dockerTools.pullImage: expose image* attributes (#41366) rust: disable tests on darwin python.pkgs.detox: 0.11 -> 0.12 certbot: disable check on darwin freeimage: fix build with glibc 2.27 shadowsocks-libev: supports darwin (#41421) pythonPackages.jira: fix build (#41419) SDL: propagate libiconv schema2ldif: Init at 1.3 nixos/exim: Add unit restart trigger (#41418) Add myself as maintainer for dnsdist Add myself as maintainer synapse-bt: init at 2018-06-04 (#41402) skim: 0.3.2 -> 0.4.0 (#41417) ctags: fix source url (#41382) ...
Motivation for this change
shadowsocks-libev is supported on darwin
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)