python*-r1.eclass: Replace non-portable use of 'declare -g'
authorMichał Górny <mgorny@gentoo.org>
Fri, 11 Dec 2015 21:26:45 +0000 (22:26 +0100)
committerMichał Górny <mgorny@gentoo.org>
Fri, 11 Dec 2015 22:42:33 +0000 (23:42 +0100)
Replace uses of 'declare -g -r' (bash-4.2+) with 'readonly'. Reported
by Arfrever. This time use split assignments followed by readonly calls
since combined calls work unreliably.

eclass/python-any-r1.eclass
eclass/python-r1.eclass
eclass/python-single-r1.eclass
eclass/python-utils-r1.eclass

index 82ead76edf1847af27ee11d895ac85cc60cc3083..721ba45206253e9a449f081e30a48cb925a35223 100644 (file)
@@ -133,7 +133,8 @@ _python_any_set_globals() {
 
                PYTHON_DEPS="${PYTHON_PKG_DEP} ${PYTHON_DEPS}"
        done
-       declare -g -r PYTHON_DEPS="|| ( ${PYTHON_DEPS})"
+       PYTHON_DEPS="|| ( ${PYTHON_DEPS})"
+       readonly PYTHON_DEPS
 }
 _python_any_set_globals
 unset -f _python_any_set_globals
index 4b9c6f36e55db377e87d874a0a639a98388acf5d..fbc39dceaa838986d9a40b96f1b9a0bafc17abaf 100644 (file)
@@ -216,8 +216,8 @@ _python_set_globals() {
        optflags+=,${flags_st[@]/%/(-)}
 
        IUSE=${flags[*]}
-       declare -g -r PYTHON_REQUIRED_USE="|| ( ${flags[*]} )"
-       declare -g -r PYTHON_USEDEP=${optflags// /,}
+       PYTHON_REQUIRED_USE="|| ( ${flags[*]} )"
+       PYTHON_USEDEP=${optflags// /,}
 
        # 1) well, python-exec would suffice as an RDEP
        # but no point in making this overcomplex, BDEP doesn't hurt anyone
@@ -232,7 +232,7 @@ _python_set_globals() {
        else
                PYTHON_DEPS+="dev-lang/python-exec:2[${PYTHON_USEDEP}]"
        fi
-       readonly PYTHON_DEPS
+       readonly PYTHON_DEPS PYTHON_REQUIRED_USE PYTHON_USEDEP
 }
 _python_set_globals
 unset -f _python_set_globals
index 158229503f99f00e54c8efbd0733604661b73d86..b8684f0240996a499c0701632cddb26d8cfb8e04 100644 (file)
@@ -238,7 +238,7 @@ _python_single_set_globals() {
                        PYTHON_DEPS+="python_single_target_${i}? ( ${PYTHON_PKG_DEP} ) "
                done
        fi
-       declare -g -r PYTHON_USEDEP=${optflags// /,}
+       PYTHON_USEDEP=${optflags// /,}
 
        # 1) well, python-exec would suffice as an RDEP
        # but no point in making this overcomplex, BDEP doesn't hurt anyone
@@ -253,7 +253,7 @@ _python_single_set_globals() {
        else
                PYTHON_DEPS+="dev-lang/python-exec:2[${PYTHON_USEDEP}]"
        fi
-       readonly PYTHON_DEPS PYTHON_REQUIRED_USE
+       readonly PYTHON_DEPS PYTHON_REQUIRED_USE PYTHON_USEDEP
 }
 _python_single_set_globals
 unset -f _python_single_set_globals
index cf0e13496245bc89b1ae4eb0d28eba7ede3f1b8f..7830323abc0522ee385fc41cb43a35aacb3eec52 100644 (file)
@@ -40,12 +40,13 @@ inherit toolchain-funcs
 # @INTERNAL
 # @DESCRIPTION:
 # All supported Python implementations, most preferred last.
-declare -g -r _PYTHON_ALL_IMPLS=(
+_PYTHON_ALL_IMPLS=(
        python2_7
        python3_3 python3_4 python3_5
        pypy pypy3
        jython2_7
 )
+readonly _PYTHON_ALL_IMPLS
 
 # @FUNCTION: _python_impl_supported
 # @USAGE: <impl>