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
coolreader: init at 3.2.45 #95923
coolreader: init at 3.2.45 #95923
Conversation
@GrahamcOfBorg eval |
src = fetchFromGitHub { | ||
owner = "adah1972"; | ||
repo = pname; | ||
rev = "libunibreak_4_3"; | ||
sha256 = "19g3ixs1ycisfdnzd8v7j5r49h0x0hshchk9qwlz4i0mjv825plx"; |
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.
to better support updates
src = fetchFromGitHub { | |
owner = "adah1972"; | |
repo = pname; | |
rev = "libunibreak_4_3"; | |
sha256 = "19g3ixs1ycisfdnzd8v7j5r49h0x0hshchk9qwlz4i0mjv825plx"; | |
src = let | |
rev_version = stdenv.lib.replaceStrings ["-"] ["_"] version; | |
in fetchFromGitHub { | |
owner = "adah1972"; | |
repo = pname; | |
rev = "libunibreak_${rev_version}"; | |
sha256 = "19g3ixs1ycisfdnzd8v7j5r49h0x0hshchk9qwlz4i0mjv825plx"; |
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.
needs to be
stdenv.lib.replaceStrings ["."] ["_"]
not -
. Getting 404 from at the moment github...
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.
Oops, apparently ofborg didn't catch this because it had already cached the download... Fixed in 2446a98
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.
Thanks @gebner!
sha256 = "0nkk4d0j04yjwanjszq8h8hvx87rnwax2k6akm4bpjxwpcs4icws"; | ||
}; | ||
|
||
nativeBuildInputs = [ cmake pkgconfig ]; |
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.
In new expressions, pkg-config
should be used instead of the pkgconfig
alias.
meta = with lib; { | ||
homepage = "https://github.com/buggins/coolreader"; | ||
description = "Cross platform open source e-book reader"; | ||
license = licenses.gpl2Only; # see https://github.com/buggins/coolreader/issues/80 |
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.
This comment says -or-later
: buggins/coolreader#80 (comment)
@GrahamcOfBorg build fbreader |
@ofborg eval |
Thanks for the reviews! |
Motivation for this change
New package. Coolreader is an e-book reader.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)