-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
pdftag: init at 1.0.4 #45666
pdftag: init at 1.0.4 #45666
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
{ stdenv, fetchFromGitHub, pkgconfig, meson, vala, ninja | ||
, gtk3, poppler, wrapGAppsHook }: | ||
|
||
stdenv.mkDerivation rec { | ||
pname = "pdftag"; | ||
name = "${pname}-${version}"; | ||
version = "1.0.4"; | ||
|
||
src = fetchFromGitHub { | ||
owner = "arrufat"; | ||
repo = pname; | ||
rev = version; | ||
sha256 = "17zk42h0n33b4p8fqlq2riqwcdi8y9m5n0ccydnk6a4x8rli97b3"; | ||
}; | ||
|
||
nativeBuildInputs = [ pkgconfig meson ninja wrapGAppsHook ]; | ||
buildInputs = [ gtk3 poppler vala ]; | ||
|
||
patchPhase = ''substituteInPlace meson.build \ | ||
--replace "install_dir: '/usr" "install_dir: '$out" | ||
''; | ||
|
||
preInstall = "mkdir -p $out/share/licenses/${pname}"; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think installing licenses is Debianism. Maybe asking upstream about disabling this might be even more important than changing the path. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @jtojnar: I filed an issue upstream. Note that in some cases the license text has to be distributed alongside the rest as part of the license itself. Installing it in $prefix seems a sane way to go. |
||
|
||
meta = with stdenv.lib; { | ||
description = "Edit metadata found in PDFs"; | ||
license = licenses.gpl3; | ||
maintainers = with maintainers; [ leenaars ]; | ||
platforms = platforms.unix; | ||
}; | ||
} |
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.
Hmm,
vala
should have gone tonativeBuildInputs
as well.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.
@jtojnar: Update with that + 1.0.5: #46633 as a new pull request as this one is closed.