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
recoll: Add support to build without QT #42876
recoll: Add support to build without QT #42876
Conversation
pkgs/top-level/all-packages.nix
Outdated
@@ -4778,7 +4778,7 @@ with pkgs; | |||
|
|||
recutils = callPackage ../tools/misc/recutils { }; | |||
|
|||
recoll = callPackage ../applications/search/recoll { }; | |||
recoll = callPackage ../applications/search/recoll { withGui = true; }; |
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.
Why not just use withGui ? true
in the arguments?
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.
no special reason, reading other examples in all-packages.nix gave me the impression it was the preferred way. I can change is you prefer it
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.
Yeah I'd prefer that, otherwise the default value in the package definition doesn't even have any use and can only confuse.
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.
Updated :)
6b3dea4
to
e7cde47
Compare
e7cde47
to
79b81c3
Compare
@GrahamcOfBorg build recoll |
Success on x86_64-linux (full log) Attempted: recoll Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: recoll Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: recoll Partial log (click to expand)
|
Thanks |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)