From patchwork Sun Jun 4 08:04:57 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bernd Kuhls X-Patchwork-Id: 770857 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3wgVvj4vMpz9sCX for ; Sun, 4 Jun 2017 18:08:49 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id E1F032F862; Sun, 4 Jun 2017 08:08:47 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 5+AVq0zvzVpm; Sun, 4 Jun 2017 08:08:40 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by silver.osuosl.org (Postfix) with ESMTP id 660682FD64; Sun, 4 Jun 2017 08:05:06 +0000 (UTC) X-Original-To: buildroot@lists.busybox.net Delivered-To: buildroot@osuosl.org Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id E57BC1C0983 for ; Sun, 4 Jun 2017 08:05:04 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id E067385A32 for ; Sun, 4 Jun 2017 08:05:04 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rkNbYCouqBlY for ; Sun, 4 Jun 2017 08:05:03 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mailout09.t-online.de (mailout09.t-online.de [194.25.134.84]) by fraxinus.osuosl.org (Postfix) with ESMTPS id 5102385A31 for ; Sun, 4 Jun 2017 08:05:03 +0000 (UTC) Received: from fwd03.aul.t-online.de (fwd03.aul.t-online.de [172.20.27.148]) by mailout09.t-online.de (Postfix) with SMTP id DF954424BC72; Sun, 4 Jun 2017 10:05:00 +0200 (CEST) Received: from fli4l.lan.fli4l (Z2HyH-ZJrhK9oQ8PZoLJnf-d3Db3mRVMxHlGQHpToOazLvk2468R4ETi0T0zpYdQB3@[79.222.103.70]) by fwd03.t-online.de with (TLSv1:ECDHE-RSA-AES256-SHA encrypted) esmtp id 1dHQWx-0oYWUi0; Sun, 4 Jun 2017 10:04:59 +0200 Received: from mahler.lan.fli4l ([192.168.1.1]:37706 helo=kuhls.lan.fli4l) by fli4l.lan.fli4l with esmtp (Exim 4.89) (envelope-from ) id 1dHQWw-0006G3-HL; Sun, 04 Jun 2017 10:04:59 +0200 From: Bernd Kuhls To: buildroot@buildroot.org Date: Sun, 4 Jun 2017 10:04:57 +0200 Message-Id: <20170604080457.7593-2-bernd.kuhls@t-online.de> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170604080457.7593-1-bernd.kuhls@t-online.de> References: <20170604080457.7593-1-bernd.kuhls@t-online.de> X-ID: Z2HyH-ZJrhK9oQ8PZoLJnf-d3Db3mRVMxHlGQHpToOazLvk2468R4ETi0T0zpYdQB3 X-TOI-MSGID: 2ff3cb1c-71f5-493f-aed9-73835e8e9596 Subject: [Buildroot] [PATCH 2/2] package/exiv2: fix gettext detection, part II X-BeenThere: buildroot@busybox.net X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Discussion and development of buildroot List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: buildroot-bounces@busybox.net Sender: "buildroot" It seems getting the gettext stuff right on all platforms is not that easy, this patch bumps exiv2 to include all gettext (and pthreads) patches currently applied upstream and adds the latest try to hopefully get things right. To reproduce the problem gettext has to be built before exiv2 so we make gettext an optional dependency when NLS support is used. Fixes http://autobuild.buildroot.net/results/a56/a56c8a8a4c76b7c359ca7f5252be93fd70c6a4c1/ Signed-off-by: Bernd Kuhls --- package/exiv2/0001-cmake-fix-gettext-support.patch | 72 ---------------------- .../exiv2/0001-cmake-simplify-libintl-check.patch | 44 +++++++++++++ ...s-depend-on-threads-not-only-in-the-xmp-c.patch | 59 ------------------ package/exiv2/exiv2.hash | 4 +- package/exiv2/exiv2.mk | 8 ++- 5 files changed, 50 insertions(+), 137 deletions(-) delete mode 100644 package/exiv2/0001-cmake-fix-gettext-support.patch create mode 100644 package/exiv2/0001-cmake-simplify-libintl-check.patch delete mode 100644 package/exiv2/0002-cmake-always-depend-on-threads-not-only-in-the-xmp-c.patch diff --git a/package/exiv2/0001-cmake-fix-gettext-support.patch b/package/exiv2/0001-cmake-fix-gettext-support.patch deleted file mode 100644 index 435e08ac6..000000000 --- a/package/exiv2/0001-cmake-fix-gettext-support.patch +++ /dev/null @@ -1,72 +0,0 @@ -From 3af0d1a6b2c12b7f62a293d19bdf235db75d6b07 Mon Sep 17 00:00:00 2001 -From: Bernd Kuhls -Date: Sat, 3 Jun 2017 17:18:24 +0200 -Subject: [PATCH] cmake: fix gettext support - -Building exiv2 with uclibc and gettext fails currently as detected -by the buildroot autobuilders: -http://autobuild.buildroot.net/results/def/defcdd3dd57b031c31bb16c0075f079a482bf52d/build-end.log - -Patch sent upstream: https://github.com/Exiv2/exiv2/pull/6 - -Signed-off-by: Bernd Kuhls ---- - config/CMakeChecks.txt | 12 ++++++++++-- - src/CMakeLists.txt | 6 +++--- - 2 files changed, 13 insertions(+), 5 deletions(-) - -diff --git a/config/CMakeChecks.txt b/config/CMakeChecks.txt -index b3a888f2..f7621a92 100644 ---- a/config/CMakeChecks.txt -+++ b/config/CMakeChecks.txt -@@ -95,8 +95,16 @@ ELSE( EXIV2_ENABLE_SHARED ) - ENDIF( EXIV2_ENABLE_SHARED ) - - IF( EXIV2_ENABLE_NLS ) -- #FIND_PACKAGE(Intl REQUIRED) -- #INCLUDE_DIRECTORIES(${Intl_INCLUDE_DIRS}) -+ FIND_PACKAGE(Intl) -+ if(Intl_FOUND) -+ INCLUDE_DIRECTORIES(${Intl_INCLUDE_DIRS}) -+ find_library(LIBINTL_LIBRARY NAMES intl libintl) -+ if(libintl_FOUND) -+ SET(LIBINTL_LIBRARIES ${LIBINTL_LIBRARY}) -+ else(libintl_FOUND) -+ SET(LIBINTL_LIBRARIES) -+ endif() -+ endif() - IF( NOT LOCALEDIR ) - SET( LOCALEDIR "${CMAKE_INSTALL_LOCALEDIR}" ) - IF( WIN32 ) -diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt -index aecd6215..e2904c6f 100644 ---- a/src/CMakeLists.txt -+++ b/src/CMakeLists.txt -@@ -316,13 +316,13 @@ msvc_runtime_configure(${EXIV2_ENABLE_SHARED} ${EXIV2_ENABLE_DYNAMIC_RUNTIME}) - # ****************************************************************************** - # exiv2 application - ADD_EXECUTABLE( exiv2 ${EXIV2_SRC} ${EXIV2_HDR} ) --TARGET_LINK_LIBRARIES( exiv2 exiv2lib ) -+TARGET_LINK_LIBRARIES( exiv2 exiv2lib ${LIBINTL_LIBRARIES} ) - INSTALL( TARGETS exiv2 ${INSTALL_TARGET_STANDARD_ARGS} ) - - # ****************************************************************************** - # connection test application - ADD_EXECUTABLE( conntest ${CONNTEST} ) --TARGET_LINK_LIBRARIES( conntest ${PRIVATE_VAR} exiv2lib ${CURL_LIBRARIES} ${SSH_LIBRARIES}) -+TARGET_LINK_LIBRARIES( conntest ${PRIVATE_VAR} exiv2lib ${CURL_LIBRARIES} ${SSH_LIBRARIES} ${LIBINTL_LIBRARIES} ) - - # ****************************************************************************** - # exifprint application -@@ -332,7 +332,7 @@ TARGET_LINK_LIBRARIES( conntest ${PRIVATE_VAR} exiv2lib ${CURL_LIBRARIES} ${SSH - # ****************************************************************************** - # remotetest application - ADD_EXECUTABLE( remotetest ${REMOTETEST} ) --TARGET_LINK_LIBRARIES( remotetest exiv2lib ) -+TARGET_LINK_LIBRARIES( remotetest exiv2lib ${LIBINTL_LIBRARIES} ) - - # ****************************************************************************** - # Headers --- -2.11.0 - diff --git a/package/exiv2/0001-cmake-simplify-libintl-check.patch b/package/exiv2/0001-cmake-simplify-libintl-check.patch new file mode 100644 index 000000000..528564f34 --- /dev/null +++ b/package/exiv2/0001-cmake-simplify-libintl-check.patch @@ -0,0 +1,44 @@ +From 3dda984c613b93eac76cf38a715f10a7bd7e4564 Mon Sep 17 00:00:00 2001 +From: Bernd Kuhls +Date: Sun, 4 Jun 2017 09:44:37 +0200 +Subject: [PATCH] cmake: simplify libintl check + +According to https://cmake.org/cmake/help/v3.4/module/FindIntl.html +the FindIntl module already checks for libintl.so. + +Patch sent upstream: https://github.com/Exiv2/exiv2/pull/8 + +Signed-off-by: Bernd Kuhls +--- + config/CMakeChecks.txt | 14 ++++---------- + 1 file changed, 4 insertions(+), 10 deletions(-) + +diff --git a/config/CMakeChecks.txt b/config/CMakeChecks.txt +index e6100c0f..0cd28ec7 100644 +--- a/config/CMakeChecks.txt ++++ b/config/CMakeChecks.txt +@@ -98,17 +98,11 @@ ENDIF( EXIV2_ENABLE_SHARED ) + + IF( EXIV2_ENABLE_NLS ) + FIND_PACKAGE(Intl) +- if(Intl_FOUND AND NOT MSVC) ++ if(Intl_FOUND) + INCLUDE_DIRECTORIES(${Intl_INCLUDE_DIRS}) +- find_library(LIBINTL_LIBRARY NAMES intl libintl) +- if(libintl_FOUND) +- SET(LIBINTL_LIBRARIES ${LIBINTL_LIBRARY}) +- else() +- SET(LIBINTL_LIBRARIES) +- if ( APPLE OR CYGWIN ) +- SET(LIBINTL_LIBRARIES -lintl) +- endif() +- endif() ++ SET(LIBINTL_LIBRARIES ${Intl_LIBRARIES}) ++ else() ++ SET(LIBINTL_LIBRARIES) + endif() + IF( NOT LOCALEDIR ) + SET( LOCALEDIR "${CMAKE_INSTALL_LOCALEDIR}" ) +-- +2.11.0 + diff --git a/package/exiv2/0002-cmake-always-depend-on-threads-not-only-in-the-xmp-c.patch b/package/exiv2/0002-cmake-always-depend-on-threads-not-only-in-the-xmp-c.patch deleted file mode 100644 index 96b966f94..000000000 --- a/package/exiv2/0002-cmake-always-depend-on-threads-not-only-in-the-xmp-c.patch +++ /dev/null @@ -1,59 +0,0 @@ -From 7acb235c05089e82b16634383ac2b33721258ba1 Mon Sep 17 00:00:00 2001 -From: Bernd Kuhls -Date: Sat, 3 Jun 2017 18:25:05 +0200 -Subject: [PATCH] cmake: always depend on threads, not only in the xmp case - -Patch sent upstream: https://github.com/Exiv2/exiv2/pull/7 - -Signed-off-by: Bernd Kuhls ---- - CMakeLists.txt | 1 - - config/CMakeChecks.txt | 2 ++ - src/CMakeLists.txt | 2 ++ - 3 files changed, 4 insertions(+), 1 deletion(-) - -diff --git a/CMakeLists.txt b/CMakeLists.txt -index 7034bb67..64604c6f 100644 ---- a/CMakeLists.txt -+++ b/CMakeLists.txt -@@ -120,7 +120,6 @@ IF( EXIV2_ENABLE_XMP ) - IF (NOT MINGW) - set(THREADS_PREFER_PTHREAD_FLAG ON) - ENDIF() -- find_package(Threads REQUIRED) - ENDIF( EXIV2_ENABLE_XMP ) - - INCLUDE( config/CMakeChecks.txt ) -diff --git a/config/CMakeChecks.txt b/config/CMakeChecks.txt -index b3a888f2..e0557642 100644 ---- a/config/CMakeChecks.txt -+++ b/config/CMakeChecks.txt -@@ -51,6 +51,8 @@ INCLUDE_DIRECTORIES( ${CMAKE_INCLUDE_PATH} ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_C - LINK_DIRECTORIES( ${CMAKE_LIBRARY_PATH} ) - SET( CMAKE_REQUIRED_INCLUDES ${CMAKE_INCLUDE_PATH} ) - -+find_package(Threads REQUIRED) -+ - IF( EXIV2_ENABLE_PNG ) - FIND_PACKAGE( ZLIB REQUIRED ) - INCLUDE_DIRECTORIES( ${ZLIB_INCLUDE_DIR} ) -diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt -index aecd6215..15d521c5 100644 ---- a/src/CMakeLists.txt -+++ b/src/CMakeLists.txt -@@ -268,10 +268,12 @@ else() - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} dl ${EXPAT_LIBRARIES} ) - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} dl ${CURL_LIBRARIES} ) - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} dl ${SSH_LIBRARIES} ) -+ TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} dl ${CMAKE_THREAD_LIBS_INIT} ) - else() - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} ${EXPAT_LIBRARIES} ) - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} ${CURL_LIBRARIES} ) - TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} ${SSH_LIBRARIES} ) -+ TARGET_LINK_LIBRARIES( exiv2lib ${PRIVATE_VAR} ${CMAKE_THREAD_LIBS_INIT} ) - endif() - endif() - --- -2.11.0 - diff --git a/package/exiv2/exiv2.hash b/package/exiv2/exiv2.hash index cdd840c22..9b7987b7a 100644 --- a/package/exiv2/exiv2.hash +++ b/package/exiv2/exiv2.hash @@ -1,4 +1,2 @@ -# From http://www.exiv2.org/download.html -md5 f936d2ca5cbe1e18c71ca2baa5e84fb4 exiv2-0.26-trunk.tar.gz # Locally calculated -sha256 0c625cbeb494aa1b9221280a5b053b54d0c9720d48fa9120cef7c6f93efd4dc3 exiv2-0.26-trunk.tar.gz +sha256 e4c40e39f2e401608f210a5efec5a458af543c73a2871b0857aec3731f002173 exiv2-4e86c1d61c4b8ab2cbbb31db0031b8e1d1872231.tar.gz diff --git a/package/exiv2/exiv2.mk b/package/exiv2/exiv2.mk index f56e86122..02bf071cd 100644 --- a/package/exiv2/exiv2.mk +++ b/package/exiv2/exiv2.mk @@ -4,9 +4,8 @@ # ################################################################################ -EXIV2_VERSION = 0.26 -EXIV2_SITE = http://www.exiv2.org/builds -EXIV2_SOURCE = exiv2-$(EXIV2_VERSION)-trunk.tar.gz +EXIV2_VERSION = 4e86c1d61c4b8ab2cbbb31db0031b8e1d1872231 +EXIV2_SITE = $(call github,Exiv2,exiv2,$(EXIV2_VERSION)) EXIV2_INSTALL_STAGING = YES EXIV2_CONF_OPTS += -DEXIV2_ENABLE_BUILD_SAMPLES=OFF @@ -41,6 +40,9 @@ endif ifeq ($(BR2_ENABLE_LOCALE),y) EXIV2_CONF_OPTS += -DEXIV2_ENABLE_NLS=ON +ifeq ($(BR2_PACKAGE_GETTEXT),y) +EXIV2_DEPENDENCIES += gettext +endif else EXIV2_CONF_OPTS += -DEXIV2_ENABLE_NLS=OFF endif