media-sound/tagtool: Fixed AC_ARG_ENABLE in configure.ac for use_enable

Package-Manager: portage-2.2.26

diff --git a/media-sound/tagtool/files/tagtool-0.12.3-underlinking.patch b/media-sound/tagtool/files/tagtool-0.12.3-underlinking.patch
index 14c4bba1eb15b259bfc8705cad2d4f765cf9ee69..9569ad2fbaf5ee7ada1ed1df323e3e7bae23f655 100644 (file)
--- a/media-sound/tagtool/files/tagtool-0.12.3-underlinking.patch
+++ b/media-sound/tagtool/files/tagtool-0.12.3-underlinking.patch
@@ -1,9 +1,40 @@
 Fix underlinking issues caused by missing libm and libogg linking
 Gentoo Bug: https://bugs.gentoo.org/show_bug.cgi?id=513012
 
+In addition, fix nonidiomatic AC_ARG_ENABLE option handling.
+
 --- tagtool-0.12.3/configure.ac
 +++ tagtool-0.12.3/configure.ac
-@@ -50,6 +50,12 @@
+@@ -13,20 +13,16 @@
+ 
+ 
+ dnl Command line options
+-AC_ARG_ENABLE(mp3,
+-      AC_HELP_STRING([--disable-mp3], [Disable MP3 support (if enabled, id3lib is required)]),
+-      [enable_mp3=no;  disable_mp3_reason="(disabled)"],
+-      [enable_mp3=yes])
+-
+-AC_ARG_ENABLE(vorbis,
+-      AC_HELP_STRING([--disable-vorbis], [Disable Ogg Vorbis support (if enabled, libvorbis is required)]),
+-      [enable_vorbis=no;  disable_vorbis_reason="(disabled)"],
+-      [enable_vorbis=yes])
++AC_ARG_ENABLE([mp3],
++      AS_HELP_STRING([--disable-mp3], [Disable MP3 support (if enabled, id3lib is required)]))
+ 
+-if test "$enable_mp3" = "no" && test "$enable_vorbis" = "no"; then
+-      AC_MSG_ERROR([At least one of 'mp3' or 'vorbis' must be enabled. Try './configure --help' for a list of options.])
+-fi;
++AC_ARG_ENABLE([vorbis],
++      AS_HELP_STRING([--disable-vorbis], [Disable Ogg Vorbis support (if enabled, libvorbis is required)]))
+ 
++dnl Test for no flags being enabled
++AS_IF([test "x$enable_mp3" != "xyes" && test "x$enable_vorbis" != "xyes" ], [
++      AC_MSG_ERROR([At least one of 'mp3' or 'vorbis' must be enabled. Try './configure --help' for a list of options.])
++])
+ 
+ dnl Checks for programs.
+ AC_PROG_CC
+@@ -50,6 +46,12 @@
  fi;
  
  
@@ -16,7 +47,7 @@ Gentoo Bug: https://bugs.gentoo.org/show_bug.cgi?id=513012
  dnl Check for gtk and related libraries
  PKG_CHECK_MODULES(GTK, [glib-2.0 >= 2.12.0
                       gtk+-2.0 >= 2.8.0
-@@ -91,6 +97,9 @@
+@@ -91,6 +93,9 @@
        AC_CHECK_LIB(vorbisfile, main,,
                [enable_vorbis=no;  disable_vorbis_reason="(missing vorbisfile library)"])
  fi;

diff --git a/media-sound/tagtool/tagtool-0.12.3-r1.ebuild b/media-sound/tagtool/tagtool-0.12.3-r1.ebuild
index 6a1d325dffed28e6cdc4eaf8d397adf74ccd1e72..7ceaa0a1dba5693f264361b21c2a13c79984f4f7 100644 (file)
--- a/media-sound/tagtool/tagtool-0.12.3-r1.ebuild
+++ b/media-sound/tagtool/tagtool-0.12.3-r1.ebuild
@@ -13,13 +13,13 @@ SRC_URI="mirror://sourceforge/${PN}/${P}.tar.bz2"
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="~amd64 ~ppc ~sparc ~x86"
-IUSE="vorbis mp3"
+IUSE="mp3 +vorbis"
+REQUIRED_USE="|| ( mp3 vorbis )"
 
 RDEPEND="x11-libs/gtk+:2
        >=gnome-base/libglade-2.6
        mp3? ( >=media-libs/id3lib-3.8.3-r6 )
-       vorbis? ( >=media-libs/libvorbis-1 )
-       !mp3? ( !vorbis? ( >=media-libs/libvorbis-1 ) )"
+       vorbis? ( >=media-libs/libvorbis-1 )"
 DEPEND="${RDEPEND}
        virtual/pkgconfig"
 
@@ -36,17 +36,9 @@ src_prepare() {
 }
 
 src_configure() {
-       local myconf
-
-       use mp3 || myconf="${myconf} --disable-mp3"
-       use vorbis || myconf="${myconf} --disable-vorbis"
-
-       if ! use mp3 && ! use vorbis; then
-               einfo "One of USE flags is required, enabling vorbis for you."
-               myconf="--disable-mp3"
-       fi
-
-       econf ${myconf}
+       econf \
+               $(use_enable mp3) \
+               $(use_enable vorbis)
 }
 
 src_install() {
@@ -54,7 +46,7 @@ src_install() {
                DESTDIR="${D}" \
                GNOME_SYSCONFDIR="${D}/etc" \
                sysdir="${D}/usr/share/applets/Multimedia" \
-               install || die
+               install
 
        dodoc ChangeLog NEWS README TODO THANKS
 }