From d309cc7c217eeb187ce59b7d5c9145112048182b Mon Sep 17 00:00:00 2001 From: Rasmus Thomsen Date: Wed, 2 Jan 2019 01:04:45 +0100 Subject: [PATCH] fix build against exiv2-0.27.x fixes #654 --- src/exiv2.cc | 45 +++++++++++++++++++++------------------------ 1 file changed, 21 insertions(+), 24 deletions(-) diff --git a/src/exiv2.cc b/src/exiv2.cc index b3c80657..2992740c 100644 --- a/src/exiv2.cc +++ b/src/exiv2.cc @@ -22,17 +22,25 @@ #ifdef HAVE_EXIV2 -#include -#include +// Don't include the file directly +// Early Exiv2 versions didn't have version.hpp and the macros. +#include #include #include // EXIV2_TEST_VERSION is defined in Exiv2 0.15 and newer. +#ifdef EXIV2_VERSION #ifndef EXIV2_TEST_VERSION -# define EXIV2_TEST_VERSION(major,minor,patch) \ +#define EXIV2_TEST_VERSION(major,minor,patch) \ ( EXIV2_VERSION >= EXIV2_MAKE_VERSION(major,minor,patch) ) #endif +#else +#define EXIV2_TEST_VERSION(major,minor,patch) (false) +#endif +#if EXIV2_TEST_VERSION(0,27,0) +#define HAVE_EXIV2_ERROR_CODE +#endif #include #include @@ -40,27 +48,8 @@ #include #include -#if !EXIV2_TEST_VERSION(0,17,90) -#include -#include -#include -#include -#include -#include -#if EXIV2_TEST_VERSION(0,16,0) -#include -#endif -#if EXIV2_TEST_VERSION(0,13,0) -#include -#endif -#include -#else -#include -#endif - -#if EXIV2_TEST_VERSION(0,17,0) -#include -#include +#if EXIV2_TEST_VERSION(0,27,0) +#define EXV_PACKAGE "exiv2" #endif extern "C" { @@ -374,7 +363,11 @@ public: #endif Exiv2::Image *image = imageData_->image(); +#ifdef HAVE_EXIV2_ERROR_CODE + if (!image) throw Exiv2::Error(Exiv2::ErrorCode::kerInputDataReadFailed); +#else if (!image) throw Exiv2::Error(21); +#endif image->setExifData(exifData_); image->setIptcData(iptcData_); #if EXIV2_TEST_VERSION(0,16,0) @@ -393,8 +386,12 @@ public: sidecar->setXmpData(xmpData_); sidecar->writeMetadata(); +#else +#ifdef HAVE_EXIV2_ERROR_CODE + throw Exiv2::Error(Exiv2::ErrorCode::kerNotAnImage, "xmp"); #else throw Exiv2::Error(3, "xmp"); +#endif #endif } } -- 2.20.1