test the actual # of args and not whether their expansion is an empty string
authorMike Frysinger <vapier@gentoo.org>
Wed, 17 Aug 2011 17:59:35 +0000 (17:59 +0000)
committerMike Frysinger <vapier@gentoo.org>
Wed, 17 Aug 2011 17:59:35 +0000 (17:59 +0000)
eclass/flag-o-matic.eclass

index e119e1be2ed0acd6d163a6b1245dd57890e38e40..dfe87bcaa827968e40f2d87b713a1ad27e8fbd99 100644 (file)
@@ -1,6 +1,6 @@
 # Copyright 1999-2009 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/eclass/flag-o-matic.eclass,v 1.153 2011/08/17 17:54:48 vapier Exp $
+# $Header: /var/cvsroot/gentoo-x86/eclass/flag-o-matic.eclass,v 1.154 2011/08/17 17:59:35 vapier Exp $
 
 # @ECLASS: flag-o-matic.eclass
 # @MAINTAINER:
@@ -130,7 +130,7 @@ filter-flags() {
 # @DESCRIPTION:
 # Remove flags that enable Large File Support.
 filter-lfs-flags() {
-       [[ -n $@ ]] && die "filter-lfs-flags takes no arguments"
+       [[ $# -ne 0 ]] && die "filter-lfs-flags takes no arguments"
        # http://www.gnu.org/s/libc/manual/html_node/Feature-Test-Macros.html
        # _LARGEFILE_SOURCE: enable support for new LFS funcs (ftello/etc...)
        # _LARGEFILE64_SOURCE: enable support for 64bit variants (off64_t/fseeko64/etc...)
@@ -183,7 +183,7 @@ append-fflags() {
 # @DESCRIPTION:
 # Add flags that enable Large File Support.
 append-lfs-flags() {
-       [[ -n $@ ]] && die "append-lfs-flags takes no arguments"
+       [[ $# -ne 0 ]] && die "append-lfs-flags takes no arguments"
        append-cppflags -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
 }