[gentoo-commits] repo/gentoo:master commit in: mail-client/mutt/

["Fabian Groffen" <grobian@gentoo.org>]

commit:     69844c31d2cd0f140dc003cbe3c6d2dee16325e7
Author:     Fabian Groffen <grobian <AT> gentoo <DOT> org>
AuthorDate: Thu Oct 13 15:59:41 2016 +0000
Commit:     Fabian Groffen <grobian <AT> gentoo <DOT> org>
CommitDate: Thu Oct 13 16:06:20 2016 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=69844c31

mail-client/mutt: cleanup econf arguments

Package-Manager: portage-2.3.0

 mail-client/mutt/mutt-1.7.1-r1.ebuild | 101 +++++++++++++++++++---------------
 1 file changed, 56 insertions(+), 45 deletions(-)

diff --git a/mail-client/mutt/mutt-1.7.1-r1.ebuild b/mail-client/mutt/mutt-1.7.1-r1.ebuild
index a9e98c7..27fe71b 100644
--- a/mail-client/mutt/mutt-1.7.1-r1.ebuild
+++ b/mail-client/mutt/mutt-1.7.1-r1.ebuild
@@ -132,74 +132,85 @@ src_prepare() {
 }
 
 src_configure() {
-	local myconf="
-		$(use_enable crypt pgp) \
-		$(use_enable debug) \
-		$(use_enable gpg gpgme) \
-		$(use_enable imap) \
-		$(use_enable nls) \
-		$(use_enable nntp) \
-		$(use_enable pop) \
-		$(use_enable sidebar) \
-		$(use_enable smime) \
-		$(use_enable smtp) \
-		$(use_enable notmuch) \
-		$(use_with idn) \
-		$(use_with kerberos gss) \
-		$(use slang && echo --with-slang=${EPREFIX}/usr) \
-		$(use !slang && echo --with-curses=${EPREFIX}/usr) \
-		--enable-compressed \
-		--enable-external-dotlock \
-		--enable-nfs-fix \
-		--sysconfdir=${EPREFIX}/etc/${PN} \
-		--with-docdir=${EPREFIX}/usr/share/doc/${PN}-${PVR} \
-		--with-regex \
-		--with-exec-shell=${EPREFIX}/bin/sh"
+	local myconf=(
+		"$(use_enable crypt pgp)"
+		"$(use_enable debug)"
+		"$(use_enable gpg gpgme)"
+		"$(use_enable imap)"
+		"$(use_enable nls)"
+		"$(use_enable nntp)"
+		"$(use_enable pop)"
+		"$(use_enable sidebar)"
+		"$(use_enable smime)"
+		"$(use_enable smtp)"
+		"$(use_enable notmuch)"
+		"$(use_with idn)"
+		"$(use_with kerberos gss)"
+		"--with-$(use slang && echo slang || echo curses)=${EPREFIX}/usr"
+		"--enable-compressed"
+		"--enable-external-dotlock"
+		"--enable-nfs-fix"
+		"--sysconfdir=${EPREFIX}/etc/${PN}"
+		"--with-docdir=${EPREFIX}/usr/share/doc/${PN}-${PVR}"
+		"--with-regex"
+		"--with-exec-shell=${EPREFIX}/bin/sh"
+	)
 
 	if [[ ${CHOST} == *-solaris* ]] ; then
 		# arrows in index view do not show when using wchar_t
-		myconf+=" --without-wc-funcs"
+		myconf+=( "--without-wc-funcs" )
 	fi
 
 	# mutt prioritizes gdbm over bdb, so we will too.
 	# hcache feature requires at least one database is in USE.
-	if use tokyocabinet; then
-		myconf="${myconf} --enable-hcache \
-			--with-tokyocabinet --without-qdbm --without-gdbm --without-bdb"
-	elif use qdbm; then
-		myconf="${myconf} --enable-hcache \
-			--without-tokyocabinet --with-qdbm --without-gdbm --without-bdb"
-	elif use gdbm ; then
-		myconf="${myconf} --enable-hcache \
-			--without-tokyocabinet --without-qdbm --with-gdbm --without-bdb"
-	elif use berkdb; then
-		myconf="${myconf} --enable-hcache \
-			--without-tokyocabinet --without-qdbm --without-gdbm --with-bdb"
+	local hcaches=(
+		"tokyocabinet"
+		"qdbm"
+		"gdbm"
+		"berkdb:bdb"
+	)
+	local ucache hcache lcache
+	for hcache in "${hcaches[@]}" ; do
+		if use ${hcache%%:*} ; then
+			ucache=${hcache}
+			break
+		fi
+	done
+	if [[ -n ${ucache} ]] ; then
+		myconf+=( "--enable-hcache" )
 	else
-		myconf="${myconf} --disable-hcache \
-			--without-tokyocabinet --without-qdbm --without-gdbm --without-bdb"
+		myconf+=( "--disable-hcache" )
 	fi
+	for hcache in "${hcaches[@]}" ; do
+		[[ ${hcache} == ${ucache} ]] \
+			&& myconf+=( "--with-${hcache#*:}" ) \
+			|| myconf+=( "--without-${hcache#*:}" )
+	done
 
 	# there's no need for gnutls, ssl or sasl without socket support
 	if use pop || use imap || use smtp ; then
 		if use gnutls; then
-			myconf="${myconf} --with-gnutls"
+			myconf+=( "--with-gnutls" )
 		elif use ssl; then
-			myconf="${myconf} --with-ssl"
+			myconf+=( "--with-ssl" )
 		fi
 		# not sure if this should be mutually exclusive with the other two
-		myconf="${myconf} $(use_with sasl)"
+		myconf+=( "$(use_with sasl)" )
 	else
-		myconf="${myconf} --without-gnutls --without-ssl --without-sasl"
+		myconf+=(
+			"--without-gnutls"
+			"--without-ssl"
+			"--without-sasl"
+		)
 	fi
 
 	if use mbox; then
-		myconf="${myconf} --with-mailpath=${EPREFIX}/var/spool/mail"
+		myconf+=( "--with-mailpath=${EPREFIX}/var/spool/mail" )
 	else
-		myconf="${myconf} --with-homespool=Maildir"
+		myconf+=( "--with-homespool=Maildir" )
 	fi
 
-	econf ${myconf} || die "configure failed"
+	econf "${myconf[@]}" || die "configure failed"
 }
 
 src_install() {