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
gimp-with-plugins: Fix build errors with exiv2-0.27.1 #64353
Conversation
@GrahamcOfBorg build gimp-with-plugins |
@@ -20,6 +20,10 @@ stdenv.mkDerivation rec { | |||
gtk2 gtkimageview bzip2 zlib | |||
libjpeg libtiff cfitsio exiv2 lcms2 lensfun | |||
] ++ stdenv.lib.optional withGimpPlugin gimp; | |||
patches = [ |
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.
Can you add a comment that these are needed for exiv2 0.27.1?
Would it also be okay to combine these patches?
#include <libgimp/gimpui.h> | ||
|
||
+#include <exiv2/error.hpp> | ||
#include <exiv2/image.hpp> |
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.
Upstream discourages from using individual header files since it breaks build, so you should just use
#include <exiv2/exiv2.hpp>
#include "ufraw.h" | ||
|
||
#ifdef HAVE_EXIV2 | ||
+#include <exiv2/error.hpp> |
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.
361fe79
to
57d8a94
Compare
Although I've updated this in response to your review comments, I expect you'll want to use #64607 instead. If so, I'll close this. |
That is true @neilmayhew. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)