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
inkscape: use latest poppler, use gtkspell2, drop -fpermissive
#72491
Conversation
Do we have a working |
Observation: |
60c4b73
to
30b466b
Compare
@7c6f434c Done. |
@GrahamcOfBorg eval |
It seems that `inkscape-0.9.4` builds with the latest poppler and without the `-fpermissive` workaround.
30b466b
to
ba0a68c
Compare
@7c6f434c Fixed another typo. |
@@ -52,7 +52,7 @@ stdenv.mkDerivation rec { | |||
librsvg # for loading icons | |||
|
|||
python2Env perlPackages.perl | |||
]; | |||
] ++ stdenv.lib.optional (!stdenv.isDarwin) gtkspell2; |
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.
Looking at ofBorg complaints, I think this check doesn't work as expected (and stdenv.isLinux
is probably a better condition)
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.
Ah, I guess isDarwin
is good enough.
It seems that
inkscape-0.9.4
builds with the latest poppler andwithout the
-fpermissive
workaround.Motivation for this change
Cleanup.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @7c6f434c (added
-fpermissive
andpoppler = poppler_0_61
); nometa.maintainer
.