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
pstoedit,plotutils: fix for darwin and mark not broken #22196
Conversation
@jbaum98, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @MarcWeber and @joachifm to be potential reviewers. |
|
||
meta = with stdenv.lib; { | ||
description = "Translates PostScript and PDF graphics into other vector formats"; | ||
homepage = https://sourceforge.net/projects/pstoedit/; | ||
license = licenses.gpl2; | ||
maintainers = [ maintainers.marcweber ]; | ||
platforms = platforms.linux; |
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.
Don't remove this line, change it to platforms.unix
@@ -13,13 +13,14 @@ stdenv.mkDerivation rec { | |||
|
|||
outputs = [ "out" "dev" ]; | |||
nativeBuildInputs = [ pkgconfig ]; | |||
buildInputs = [ zlib ghostscript imagemagick plotutils gd libjpeg libwebp ]; | |||
buildInputs = [ zlib ghostscript imagemagick plotutils gd libjpeg libwebp libiconv ] ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [ |
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.
I'd prefer having this line wrapped earlier
@@ -52,6 +52,5 @@ stdenv.mkDerivation rec { | |||
|
|||
license = stdenv.lib.licenses.gpl2Plus; | |||
maintainers = [ stdenv.lib.maintainers.marcweber ]; | |||
platforms = stdenv.lib.platforms.gnu; |
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.
Don't remove this line, change it to platforms.unix
@globin thanks for the suggestions, I made those changes and pushed a new commit. |
@@ -13,13 +13,16 @@ stdenv.mkDerivation rec { | |||
|
|||
outputs = [ "out" "dev" ]; | |||
nativeBuildInputs = [ pkgconfig ]; | |||
buildInputs = [ zlib ghostscript imagemagick plotutils gd libjpeg libwebp ]; | |||
buildInputs = [ zlib ghostscript imagemagick plotutils gd libjpeg libwebp libiconv ] |
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.
libiconv is only needed on dariwn.
Motivation for this change
pstoedit and plotutils were marked broken for darwin, and pstoedit was not compiling due to a missing dependency on ApplicationServices.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)