From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp0 ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms11 with LMTPS id QOLQDKPNHF+xBwAA0tVLHw (envelope-from ) for ; Sun, 26 Jul 2020 00:26:11 +0000 Received: from aspmx1.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp0 with LMTPS id OMOhCKPNHF9zKgAA1q6Kng (envelope-from ) for ; Sun, 26 Jul 2020 00:26:11 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 868579403CB for ; Sun, 26 Jul 2020 00:26:09 +0000 (UTC) Received: from localhost ([::1]:53678 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jzUUR-00050W-Ae for larch@yhetil.org; Sat, 25 Jul 2020 20:26:07 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:54414) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jzUUM-000506-VN for guix-patches@gnu.org; Sat, 25 Jul 2020 20:26:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:39821) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jzUUM-0001go-MG for guix-patches@gnu.org; Sat, 25 Jul 2020 20:26:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jzUUM-0008G6-J2 for guix-patches@gnu.org; Sat, 25 Jul 2020 20:26:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#42542] [PATCH] gnu: libextractor: Update to 1.10. Resent-From: Michael Rohleder Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sun, 26 Jul 2020 00:26:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 42542 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 42542@debbugs.gnu.org X-Debbugs-Original-To: guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.159572316031736 (code B ref -1); Sun, 26 Jul 2020 00:26:02 +0000 Received: (at submit) by debbugs.gnu.org; 26 Jul 2020 00:26:00 +0000 Received: from localhost ([127.0.0.1]:51367 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jzUUE-0008Fi-1M for submit@debbugs.gnu.org; Sat, 25 Jul 2020 20:26:00 -0400 Received: from lists.gnu.org ([209.51.188.17]:56100) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jzUU9-0008FZ-GI for submit@debbugs.gnu.org; Sat, 25 Jul 2020 20:25:53 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:54322) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jzUU9-0004yT-97 for guix-patches@gnu.org; Sat, 25 Jul 2020 20:25:49 -0400 Received: from wp224.webpack.hosteurope.de ([2a01:488:42:1000:50ed:84e7::]:41234) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jzUU3-0001WP-Ph for guix-patches@gnu.org; Sat, 25 Jul 2020 20:25:48 -0400 Received: from www.rohleder.de ([37.61.204.227]); authenticated by wp224.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) id 1jzUTy-0000ID-AC; Sun, 26 Jul 2020 02:25:38 +0200 Received: from [192.168.1.3] (helo=micha) by www.rohleder.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94) (envelope-from ) id 1jzUTw-0003zT-Kg for guix-patches@gnu.org; Sun, 26 Jul 2020 02:25:37 +0200 From: Michael Rohleder X-PGP-Fingerprint: 755E 2DE5 D0D5 85C5 2E78 2830 7C7A FFBE FEF2 CB25 X-PGP-Key: https://www.rohleder.de/~mike/pgpkey.asc Date: Sun, 26 Jul 2020 02:25:29 +0200 Message-ID: <87h7tv5dhi.fsf@rohleder.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/signed; boundary="==-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" X-bounce-key: webpack.hosteurope.de;mike@rohleder.de;1595723143;8e4c754e; X-HE-SMSGID: 1jzUTy-0000ID-AC Received-SPF: permerror client-ip=2a01:488:42:1000:50ed:84e7::; envelope-from=mike@rohleder.de; helo=wp224.webpack.hosteurope.de X-detected-operating-system: by eggs.gnu.org: First seen = 2020/07/25 20:25:38 X-ACL-Warn: Detected OS = ??? X-Spam_score_int: -8 X-Spam_score: -0.9 X-Spam_bar: / X-Spam_report: (-0.9 / 5.0 requ) BAYES_00=-1.9, HEXHASH_WORD=1, SPF_HELO_NONE=0.001, T_SPF_PERMERROR=0.01, URIBL_BLOCKED=0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-Spam-Score: -2.3 (--) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-Spam-Score: -1.0 (-) X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Scanner: scn0 Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Spam-Score: -1.61 X-TUID: Btzc1VLWyq2c --==-=-= Content-Type: multipart/mixed; boundary="=-=-=" --=-=-= Content-Type: text/plain Dear guix, After reading about the exciting dkgpg pkg, I wanted to try gnunet the other day and noticed that libextractor doesnt build, because of some test failures in exiv2 tests: --=-=-= Content-Type: text/plain Content-Disposition: inline ... ================================================= libextractor 1.9: src/plugins/test-suite.log ================================================== # TOTAL: 29 # PASS: 28 # SKIP: 0 # XFAIL: 0 # FAIL: 1 # XPASS: 0 # ERROR: 0 .. contents:: :depth: 2 FAIL: test_exiv2 ================ Warning: Directory Pentax, entry 0x0004: Data area exceeds data buffer, ignoring it. Got additional meta data of type 38 and format 1 with value `28deg 8' 18"' from plugin `exiv2' Got additional meta data of type 40 and format 1 with value `14deg 14' 22"' from plugin `exiv2' Did not get expected meta data of type 38 and format 1 with value `28deg 8' 17.585" ' from plugin `exiv2' Did not get expected meta data of type 40 and format 1 with value `14deg 14' 21.713" ' from plugin `exiv2' FAIL test_exiv2 (exit status: 1) --=-=-= Content-Type: text/plain It seems, commit 630b75f76a did broke something here. But because we also have exiv2-0.26, I think we could use that and drop the patch for exiv2-0.27 (which would not only needs to be modified if libextractor changes, but also if exiv2 changes?!). I removed tidy-html, because its not realy recognized by configure (also in 1.9, I think) and the configure-flag "--with-tidy" was dropped in 1.10. giflib on the other hand seems to work in 1.10, tests are ok and the libs for it is generated. I am not sure if its ok to do all this in one patch, I think I could also make 2 or 3 patches (fix build, update to 1.10, update inputs or so), but I hope the intention is clear (and gnunet builds, which it does) ;) --=-=-= Content-Type: text/x-patch; charset=utf-8 Content-Disposition: inline; filename=0001-gnu-libextractor-Update-to-1.10.patch Content-Transfer-Encoding: quoted-printable Content-Description: [PATCH] gnu: libextractor: Update to 1.10. * gnu/packages/gnunet.scm (libextractor): Update to 1.10. [inputs]: commit 630b75f76a broke exiv2 test, so build with exiv2-0.26. [inputs]: added giflib, removed tidy-html * gnu/packages/patches/libextractor-exiv2.patch removed. * gnu/local.mk: remove libextractor-exiv2.patch. =2D-- gnu/local.mk | 1 - gnu/packages/gnunet.scm | 17 +-- gnu/packages/patches/libextractor-exiv2.patch | 124 ------------------ 3 files changed, 7 insertions(+), 135 deletions(-) delete mode 100644 gnu/packages/patches/libextractor-exiv2.patch diff --git a/gnu/local.mk b/gnu/local.mk index f2a7b6b984..5c92db2e05 100644 =2D-- a/gnu/local.mk +++ b/gnu/local.mk @@ -1207,7 +1207,6 @@ dist_patch_DATA =3D \ %D%/packages/patches/libbonobo-activation-test-race.patch \ %D%/packages/patches/libcanberra-sound-theme-freedesktop.patch \ %D%/packages/patches/libdrm-realpath-virtio.patch \ =2D %D%/packages/patches/libextractor-exiv2.patch \ %D%/packages/patches/libgeotiff-adapt-test-script-for-proj-6.2.patch \ %D%/packages/patches/libgit2-mtime-0.patch \ %D%/packages/patches/libgnome-encoding.patch \ diff --git a/gnu/packages/gnunet.scm b/gnu/packages/gnunet.scm index 979dadbe2e..77dc71a958 100644 =2D-- a/gnu/packages/gnunet.scm +++ b/gnu/packages/gnunet.scm @@ -10,6 +10,7 @@ ;;; Copyright =C2=A9 2018 Alex Vong ;;; Copyright =C2=A9 2019 Brett Gilio ;;; Copyright =C2=A9 2020 Tanguy Le Carrour +;;; Copyright =C2=A9 2020 Michael Rohleder ;;; ;;; This file is part of GNU Guix. ;;; @@ -73,31 +74,30 @@ (define-public libextractor (package (name "libextractor") =2D (version "1.9") + (version "1.10") (source (origin (method url-fetch) (uri (string-append "mirror://gnu/libextractor/libextractor-" version ".tar.gz")) (sha256 (base32 =2D "1zz2zvikvfibxnk1va3kgzs7djsmiqy7bmk8y01vbsf54ryjb3zh")) =2D (patches (search-patches "libextractor-exiv2.patch")))) + "0mr38g7kfn3p050hd3hckbcz2yd3za6dwl1c26x2kjf7vnsi3vcy")))) (build-system gnu-build-system) ;; WARNING: Checks require /dev/shm to be in the build chroot, especial= ly ;; not to be a symbolic link to /run/shm. ;; FIXME: ;; The following dependencies are all optional, but should be ;; available for maximum coverage: =2D ;; * libmagic (file) ;; * librpm (rpm) ; investigate failure =2D ;; * libgif (giflib) ; investigate failure + ;; * libtidy-html (tidy-html) ; investigate failure (inputs =2D `(("exiv2" ,exiv2) + `(("exiv2" ,exiv2-0.26) ("bzip2" ,bzip2) ("flac" ,flac) ("ffmpeg" ,ffmpeg) ("file" ,file) ;libmagic, for the MIME plu= g-in ("glib" ,glib) + ("giflib" ,giflib) ("gstreamer" ,gstreamer) ("gst-plugins-base" ,gst-plugins-base) ("gtk+" ,gtk+) @@ -108,7 +108,6 @@ ("libmpeg2" ,libmpeg2) ("libmp4v2" ,libmp4v2) ("libsmf" ,libsmf) =2D ("tidy-html" ,tidy-html) ("libogg" ,libogg) ("libtiff" ,libtiff) ("libvorbis" ,libvorbis) @@ -120,9 +119,7 @@ (arguments `(#:configure-flags (list (string-append "--with-ltdl=3D" =2D (assoc-ref %build-inputs "libltdl")) =2D (string-append "--with-tidy=3D" =2D (assoc-ref %build-inputs "tidy-html"))) + (assoc-ref %build-inputs "libltdl"))) #:parallel-tests? #f #:phases (modify-phases %standard-phases diff --git a/gnu/packages/patches/libextractor-exiv2.patch b/gnu/packages/p= atches/libextractor-exiv2.patch deleted file mode 100644 index b92fef3ca1..0000000000 =2D-- a/gnu/packages/patches/libextractor-exiv2.patch +++ /dev/null @@ -1,124 +0,0 @@ =2DThis patch allows us to build libextractor against exiv2 0.27.x. =2DAdapted from this upstream commit: =2D =2D commit 1ecee9a47717e36cb8a3925d011d1a6de11d631c =2D Author: Christian Grothoff =2D Date: Mon Jul 29 17:58:18 2019 +0200 =2D =2D importing patch from Gentoo/AS to address exiv2 build issue (#5820) =2D =2Ddiff --git a/src/plugins/exiv2_extractor.cc b/src/plugins/exiv2_extracto= r.cc =2Dindex 8741d40..ef402a8 100644 =2D--- a/src/plugins/exiv2_extractor.cc =2D+++ b/src/plugins/exiv2_extractor.cc =2D@@ -27,10 +27,7 @@ =2D #include =2D #include =2D #include =2D-#include =2D-#include =2D-#include =2D-#include =2D+#include =2D=20 =2D /** =2D * Enable debugging to get error messages. =2D@@ -180,7 +177,7 @@ public: =2D * =2D * @return -1 on error =2D */ =2D-#if EXIV2_VERSION >=3D EXIV2_MAKE_VERSION(0,26,0) =2D+#if EXIV2_TEST_VERSION(0,26,0) =2D virtual size_t size (void) const; =2D #else =2D virtual long int size (void) const; =2D@@ -316,7 +313,11 @@ ExtractorIO::getb () =2D const unsigned char *r; =2D=20 =2D if (1 !=3D ec->read (ec->cls, &data, 1)) =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D r =3D (const unsigned char *) data; =2D return *r; =2D } =2D@@ -371,7 +372,11 @@ ExtractorIO::putb (Exiv2::byte data) =2D void =2D ExtractorIO::transfer (Exiv2::BasicIo& src) =2D { =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D } =2D=20 =2D=20 =2D@@ -416,7 +421,11 @@ ExtractorIO::seek (long offset, =2D Exiv2::byte * =2D ExtractorIO::mmap (bool isWritable) =2D { =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D } =2D=20 =2D=20 =2D@@ -449,7 +458,7 @@ ExtractorIO::tell (void) const =2D * =2D * @return -1 on error =2D */ =2D-#if EXIV2_VERSION >=3D EXIV2_MAKE_VERSION(0,26,0) =2D+#if EXIV2_TEST_VERSION(0,26,0) =2D size_t =2D #else =2D long int =2D@@ -504,7 +513,11 @@ ExtractorIO::eof () const =2D std::string =2D ExtractorIO::path () const =2D { =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D } =2D=20 =2D=20 =2D@@ -517,7 +530,11 @@ ExtractorIO::path () const =2D std::wstring =2D ExtractorIO::wpath () const =2D { =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D } =2D #endif =2D=20 =2D@@ -531,7 +548,11 @@ Exiv2::BasicIo::AutoPtr =2D ExtractorIO::temporary () const =2D { =2D fprintf (stderr, "throwing temporary error\n"); =2D+#if EXIV2_TEST_VERSION(0,27,0) =2D+ throw Exiv2::BasicError (Exiv2::kerDecodeLangAltQualifierFail= ed); =2D+#else =2D throw Exiv2::BasicError (42 /* error code */); =2D+#endif =2D } =2D=20 =2D=20 =2D@@ -697,7 +718,7 @@ EXTRACTOR_exiv2_extract_method (struct EXTRACTOR_Ext= ractContext *ec) =2D { =2D try =2D { =2D-#if EXIV2_MAKE_VERSION(0,23,0) <=3D EXIV2_VERSION =2D+#if !EXIV2_TEST_VERSION(0,24,0) =2D Exiv2::LogMsg::setLevel (Exiv2::LogMsg::mute); =2D #endif =2D std::auto_ptr eio(new ExtractorIO (ec)); =2D-=20 2.27.0 --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable =2D-=20 The German opposite of 'umfahren' (running something over) is 'umfahren' (drive around something). Good luck mastering this language. --=-=-=-- --==-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEdV4t5dDVhcUueCgwfHr/vv7yyyUFAl8czXkACgkQfHr/vv7y yyVfDQf/SbgsYBPqwrXmDtFs6kBqAXQXT4yu6BmhfKPurIC2lS93ilyx/GebbwFN e5ToEQ/4uRQOj9HCfqtp2V0iKEv9kLLkxFJbWcFIGJ/VnmTL8S4GYlIGIfxUPdqD mvYOjCkQxot7hQZJJM/OXz/ziS9c9NjSM6f4gsytNyTvwxfF9I2G4OUmZYWARUFr rydk9+fFzx92I2YjuCwm/ReP6ahBDszkmaKIfaMsEIPAt/oFE8+zmSqkNbIteD/A ohDNnOVBOZeemH0H1eSzYmxX+s570Xjv0qcjsPfmccP2178l0jWS7xHQdOBzzUda 7nzhN9jrfJIhn3QjHglLHn/EMv1wEA== =Rbg8 -----END PGP SIGNATURE----- --==-=-=--