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
soulseekqt: init at 2016-1-17 (last official stable) #36343
Conversation
name = "soulseekqt-${version}"; | ||
version = "2016-1-17"; | ||
|
||
mainbin = "SoulseekQt-" + (version) +"-"+ (if stdenv.is64bit then "64bit" else "32bit"); |
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.
Could you move this attribute out of the derivation attributes (by using let ... in mkDerivation ...
) since it is not used by the builder.
version = "2016-1-17"; | ||
|
||
mainbin = "SoulseekQt-" + (version) +"-"+ (if stdenv.is64bit then "64bit" else "32bit"); | ||
src = fetchurl ({ |
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 think readability could be improved by moving some parts of this in the let block.
|
||
setSourceRoot = "sourceRoot=`pwd`"; | ||
|
||
libPath = stdenv.lib.makeLibraryPath |
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.
Not required by the builder also.
Thanks for these improvments:) @GrahamcOfBorg build soulseekqt |
Failure on x86_64-linux (full log) Partial log (click to expand)
|
Failure on aarch64-linux (full log) Partial log (click to expand)
|
Could you squash your commits and rewrite the first commit since it is no longer a "wip" commit. |
sorry i was trying to squash commit and look for some deeper test since i don't see any Qt in ldd, i fear it call it dynamically from my non-nixos path. |
qt is hardlinked, so i think we are good now. Thanks for review. |
Ok. It've tryied and it works well on my NixOS machine. Indeed, it can't use libraries from your host since it doen't use the ld program of your host. |
@nlewo : done. I use rebase + "push -f", i'm not very satisfied with this workflow. |
@bignaux since it's nice to have a clean history for public branches, you need to rewrite the history of your private branch and then you must Thanks for your contribution. |
soulseekqt: init at 2016-1-17 (last official stable) (cherry picked from commit 9cfeabb)
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)