net-proxy/haproxy: Use an array to define emake arguments
authorBertrand Jacquin <bertrand@jacquin.bzh>
Sat, 17 Sep 2016 23:38:51 +0000 (00:38 +0100)
committerChristian Ruppert <idl0r@gentoo.org>
Wed, 28 Sep 2016 19:53:50 +0000 (21:53 +0200)
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@gentoo.org>
net-proxy/haproxy/haproxy-1.6.9.ebuild
net-proxy/haproxy/haproxy-9999.ebuild

index 85181352dc54d0e09ac31d61d48a7305bda32dad..3052698c1e31bc48bb16afcdd44ecf8b87ccf761 100644 (file)
@@ -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
 }
index f2325eb566e5ca638a9f62956cf3364abb008265..a140ddb0af61e16ed8dbd11987f41e637f6c1545 100644 (file)
@@ -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
 }