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
nomacs: init at 3.4 #24580
nomacs: init at 3.4 #24580
Conversation
|
||
patches = [./fix-appdata-install.patch]; | ||
|
||
buildInputs = [cmake |
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.
Cmake pkgconfig and makeWrapper belong in nativeBuildInputs
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.
OK, done.
stdenv.mkDerivation rec { | ||
version = "3.4"; | ||
src = fetchurl { | ||
url = "https://github.com/nomacs/nomacs/archive/${version}.tar.gz"; |
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.
We prefer fetchFromGitHub
usually here.
src = fetchurl { | ||
url = "https://github.com/nomacs/nomacs/archive/${version}.tar.gz"; | ||
sha256 = "552eda88aedea48831ce354095e3aad47892b4b5029f424171bedb68271c2a2f"; | ||
sha512 = "67a1b57971dc373d5a3be75b7deaff6702893252568eef135903754b2465416a58b40f18f55cf2994c8c3853ae96b82506c1caf26b0e645c20179a9cd81c0d36"; |
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.
There is no need for having both checksum. Only one checksum is evaluated at the moment.
Motivation for this change
Add nomacs-3.4, a Qt-based image viewer.
Things done
nix-shell -p nox --run "nox-review wip"
./result/bin/
)