Bug 489918 – media-libs/chromaprint-1.0 version bump

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Emery Hemingway 2013-10-30 23:08:34 UTC]

Chromaprint is now at 1.0

Picard, which optionally depends on Chromaprint, needs version 1.0.

Reproducible: Always

[Comment 1 Emery Hemingway 2013-10-30 23:10:30 UTC]

Created attachment 362330 [details]
chromaprint-1.0.ebuild

Still needs one of the patches from -0.7, sci-libs/fftw:3 dependency -> sci-libs/fftw:3.0.

[Comment 2 Emery Hemingway 2013-10-30 23:11:19 UTC]

Created attachment 362332 [details, diff]
chromaprint-1.0-ffmpeg.patch

[Comment 3 Emery Hemingway 2013-10-30 23:20:43 UTC]

This bug blocks https://bugs.gentoo.org/show_bug.cgi?id=489920

[Comment 4 Jeroen Roovers (RETIRED) 2013-10-31 14:42:30 UTC]

Comment on attachment 362330 [details]
chromaprint-1.0.ebuild

--- chromaprint-0.7.ebuild      2013-04-06 19:05:15.582958160 +0200
+++ -   2013-10-31 15:42:21.795005023 +0100
@@ -1,22 +1,24 @@
 # Copyright 1999-2013 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/media-libs/chromaprint/chromaprint-0.7.ebuild,v 1.6 2013/04/05 11:04:22 ago Exp $
+# $Header: $
 
-EAPI=4
+EAPI=5
 inherit cmake-utils
 
 DESCRIPTION="A client-side library that implements a custom algorithm for extracting fingerprints"
 HOMEPAGE="http://acoustid.org/chromaprint"
-SRC_URI="mirror://github/lalinsky/${PN}/${P}.tar.gz"
+SRC_URI="https://bitbucket.org/acoustid/chromaprint/downloads/${P}.tar.gz"
 
 LICENSE="LGPL-2.1"
 SLOT="0"
-KEYWORDS="amd64 ~ppc x86"
-IUSE="test tools"
+KEYWORDS="~amd64 ~ppc ~x86"
+IUSE="test tools +ffmpeg fftw"
 
 # note: use ffmpeg instead of fftw because it's recommended and required for tools
-RDEPEND=">=virtual/ffmpeg-0.10
-       tools? ( >=media-libs/taglib-1.6 )"
+RDEPEND="
+       ffmpeg? ( >=virtual/ffmpeg-0.10 )
+       fftw?   ( sci-libs/fftw:3.0 )
+       tools?  ( >=media-libs/taglib-1.6 )"
 DEPEND="${RDEPEND}
        test? (
                dev-cpp/gtest
@@ -26,9 +28,7 @@
 
 DOCS="NEWS.txt README.txt"
 
-PATCHES=( "${FILESDIR}"/${P}-boost.patch
-                 "${FILESDIR}"/${P}-ffmpeg.patch
-                 "${FILESDIR}"/${P}-libav9.patch )
+PATCHES=( "${FILESDIR}"/${P}-ffmpeg.patch )
 
 src_configure() {
        local mycmakeargs=(

[Comment 5 Ben de Groot (RETIRED) 2013-11-03 09:41:19 UTC]

It also still needed the last hunk of the libav9 patch. And since the ffmpeg patch didn't change, there was no need to attach it separately -- we can use the existing one.