[gentoo-commits] repo/gentoo:master commit in: net-proxy/haproxy/

["Christian Ruppert" <idl0r@gentoo.org>]

commit:     a37afa92e2fcdc3c768f162480edb70143a937a7
Author:     Bertrand Jacquin <bertrand <AT> jacquin <DOT> bzh>
AuthorDate: Sat Sep 17 23:38:51 2016 +0000
Commit:     Christian Ruppert <idl0r <AT> gentoo <DOT> org>
CommitDate: Wed Sep 28 19:53:50 2016 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a37afa92

net-proxy/haproxy: Use an array to define emake arguments

It makes the code more clear and avoid duplication

Package-Manager: portage-2.2.28
Closes: https://github.com/gentoo/gentoo/pull/2293

Signed-off-by: Christian Ruppert <idl0r <AT> gentoo.org>

 net-proxy/haproxy/haproxy-1.6.9.ebuild | 41 ++++++++++++++++++----------------
 net-proxy/haproxy/haproxy-9999.ebuild  | 41 ++++++++++++++++++----------------
 2 files changed, 44 insertions(+), 38 deletions(-)

diff --git a/net-proxy/haproxy/haproxy-1.6.9.ebuild b/net-proxy/haproxy/haproxy-1.6.9.ebuild
index 8518135..3052698 100644
--- a/net-proxy/haproxy/haproxy-1.6.9.ebuild
+++ b/net-proxy/haproxy/haproxy-1.6.9.ebuild
@@ -55,66 +55,69 @@ src_prepare() {
 }
 
 src_compile() {
-	local args="TARGET=linux2628 USE_GETADDRINFO=1"
+	local -a args=(
+		TARGET=linux2628
+		USE_GETADDRINFO=1
+	)
 
 	if use crypt ; then
-		args="${args} USE_LIBCRYPT=1"
+		args+=( USE_LIBCRYPT=1 )
 	else
-		args="${args} USE_LIBCRYPT="
+		args+=( USE_LIBCRYPT= )
 	fi
 
 # bug 541042
 #	if use lua; then
-#		args="${args} USE_LUA=1"
+#		args+=( USE_LUA=1 )
 #	else
-		args="${args} USE_LUA="
+		args+=( USE_LUA= )
 #	fi
 
 	if use net_ns; then
-		args="${args} USE_NS=1"
+		args+=( USE_NS=1 )
 	else
-		args="${args} USE_NS="
+		args+=( USE_NS= )
 	fi
 
 	if use pcre ; then
-		args="${args} USE_PCRE=1"
+		args+=( USE_PCRE=1 )
 	else
-		args="${args} USE_PCRE="
+		args+=( USE_PCRE= )
 	fi
 
 	if use pcre-jit; then
-		args="${args} USE_PCRE_JIT=1"
+		args+=( USE_PCRE_JIT=1 )
 	else
-		args="${args} USE_PCRE_JIT="
+		args+=( USE_PCRE_JIT= )
 	fi
 
 #	if use kernel_linux; then
-#		args="${args} USE_LINUX_SPLICE=1 USE_LINUX_TPROXY=1"
+#		args+=( USE_LINUX_SPLICE=1 USE_LINUX_TPROXY=1 )
 #	else
-#		args="${args} USE_LINUX_SPLICE= USE_LINUX_TPROXY="
+#		args+=( USE_LINUX_SPLICE= USE_LINUX_TPROXY= )
 #	fi
 
 	if use ssl ; then
-		args="${args} USE_OPENSSL=1"
+		args+=( USE_OPENSSL=1 )
 	else
-		args="${args} USE_OPENSSL="
+		args+=( USE_OPENSSL= )
 	fi
 
 	if use zlib ; then
-		args="${args} USE_ZLIB=1"
+		args+=( USE_ZLIB=1 )
 	else
-		args="${args} USE_ZLIB="
+		args+=( USE_ZLIB= )
 	fi
 
 	# For now, until the strict-aliasing breakage will be fixed
 	append-cflags -fno-strict-aliasing
 
-	emake CFLAGS="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args}
+	emake CFLAGS="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args[@]}
 
 	if use tools ; then
 		for contrib in halog iprange ; do
 			emake -C contrib/${contrib} \
-				CFLAGS="${CFLAGS}" OPTIMIZE="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args}
+				CFLAGS="${CFLAGS}" OPTIMIZE="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args[@]}
 		done
 	fi
 }

diff --git a/net-proxy/haproxy/haproxy-9999.ebuild b/net-proxy/haproxy/haproxy-9999.ebuild
index f2325eb..a140ddb 100644
--- a/net-proxy/haproxy/haproxy-9999.ebuild
+++ b/net-proxy/haproxy/haproxy-9999.ebuild
@@ -55,66 +55,69 @@ src_prepare() {
 }
 
 src_compile() {
-	local args="TARGET=linux2628 USE_GETADDRINFO=1"
+	local -a args=(
+		TARGET=linux2628
+		USE_GETADDRINFO=1
+	)
 
 	if use crypt ; then
-		args="${args} USE_LIBCRYPT=1"
+		args+=( USE_LIBCRYPT=1 )
 	else
-		args="${args} USE_LIBCRYPT="
+		args+=( USE_LIBCRYPT= )
 	fi
 
 # bug 541042
 #	if use lua; then
-#		args="${args} USE_LUA=1"
+#		args+=( USE_LUA=1 )
 #	else
-		args="${args} USE_LUA="
+		args+=( USE_LUA= )
 #	fi
 
 	if use net_ns; then
-		args="${args} USE_NS=1"
+		args+=( USE_NS=1 )
 	else
-		args="${args} USE_NS="
+		args+=( USE_NS= )
 	fi
 
 	if use pcre ; then
-		args="${args} USE_PCRE=1"
+		args+=( USE_PCRE=1 )
 	else
-		args="${args} USE_PCRE="
+		args+=( USE_PCRE= )
 	fi
 
 	if use pcre-jit; then
-		args="${args} USE_PCRE_JIT=1"
+		args+=( USE_PCRE_JIT=1 )
 	else
-		args="${args} USE_PCRE_JIT="
+		args+=( USE_PCRE_JIT= )
 	fi
 
 #	if use kernel_linux; then
-#		args="${args} USE_LINUX_SPLICE=1 USE_LINUX_TPROXY=1"
+#		args+=( USE_LINUX_SPLICE=1 USE_LINUX_TPROXY=1 )
 #	else
-#		args="${args} USE_LINUX_SPLICE= USE_LINUX_TPROXY="
+#		args+=( USE_LINUX_SPLICE= USE_LINUX_TPROXY= )
 #	fi
 
 	if use ssl ; then
-		args="${args} USE_OPENSSL=1"
+		args+=( USE_OPENSSL=1 )
 	else
-		args="${args} USE_OPENSSL="
+		args+=( USE_OPENSSL= )
 	fi
 
 	if use zlib ; then
-		args="${args} USE_ZLIB=1"
+		args+=( USE_ZLIB=1 )
 	else
-		args="${args} USE_ZLIB="
+		args+=( USE_ZLIB= )
 	fi
 
 	# For now, until the strict-aliasing breakage will be fixed
 	append-cflags -fno-strict-aliasing
 
-	emake CFLAGS="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args}
+	emake CFLAGS="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args[@]}
 
 	if use tools ; then
 		for contrib in halog iprange ; do
 			emake -C contrib/${contrib} \
-				CFLAGS="${CFLAGS}" OPTIMIZE="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args}
+				CFLAGS="${CFLAGS}" OPTIMIZE="${CFLAGS}" LDFLAGS="${LDFLAGS}" CC=$(tc-getCC) ${args[@]}
 		done
 	fi
 }