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
Add qcomicbook. #44074
Add qcomicbook. #44074
Conversation
@GrahamcOfBorg build qcomicbook Hi! first PR there :) grats! @GrahamcOfBorg is our CI pipeline, builds can be queued and thus tested this way (you need permission first for builds). You will soon see the results on each platforms for the build. |
Success on x86_64-linux (full log) Attempted: qcomicbook Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: qcomicbook Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: qcomicbook Partial log (click to expand)
|
with stdenv.lib; | ||
|
||
let | ||
version = "0.9.1"; |
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 best practice to define the version
like this:
stdenv.mkDerivation rec {
name = "qcomicbook-${version}";
version = "0.9.1";
...
}
description = "Comic book reader in Qt5"; | ||
license = licenses.gpl2; | ||
|
||
longDescription = ''QComicBook is a viewer for PDF files and comic book archives containing |
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.
Nit: The preferred formatting is:
longDescription = ''
QComicBook is a viewer...
'';
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.
Fixed.
I've added a few minor changes that you might want to copy and then it looks about perfect: Welcome to |
Neat! I didn't know about fetchFromGithub. Thanks! |
@GrahamcOfBorg build qcomicbook |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: qcomicbook Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qcomicbook Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: qcomicbook 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.
Please modify according to nixpkgs conventions for qt applications.
@xeji done. Sorry that it took this long. |
Thank you! |
Success on x86_64-linux (full log) Attempted: qcomicbook Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: qcomicbook Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: qcomicbook Partial log (click to expand)
|
Motivation for this change
Having this program in NixOS.
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)