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
kiwix: refactor and fix expression #23525
Conversation
@@ -28,6 +28,7 @@ let | |||
then { tar = xulrunner64_tar; sdk = xulrunnersdk64_tar; } | |||
else { tar = xulrunner32_tar; sdk = xulrunnersdk32_tar; }; | |||
|
|||
ctpp2 = callPackage ../../../development/libraries/ctpp2 { inherit stdenv; }; |
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.
Sorry I do not understand the purpose of using callPackage
here. How is stdenv in this context different?
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.
From all-packages.nix
:
kiwix = callPackage ../applications/misc/kiwix {
stdenv = overrideCC stdenv gcc49;
};
Also related to #23253 |
kiwix appears to still be broken after this PR?
|
Graham Christensen <notifications@github.com> writes:
kiwix appears to still be broken after this PR?
error: cannot coerce a set to a string, at /home/grahamc/projects/nixpkgs/pkgs/applications/misc/kiwix/default.nix:64:3
Are you sure?
~~~~
$ git checkout pkg/pretty-kiwix ; nix-build ./default.nix -A kiwix
Already on 'pkg/pretty-kiwix'
Your branch is ahead of 'upstream/master' by 2 commits.
(use "git push" to publish your local commits)
/nix/store/nfpmd21qawljr4m9bykml3ays5b6pb5s-kiwix-0.9
~~~~
Works for me.
|
Additional fix in 09d7f7d |
Motivation for this change
OCD. Also doesn't build properly without patching
ctpp2
's stdenv.Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)