diff options
| author |   Ulrich Müller <ulm@gentoo.org> | 2019-11-23 11:18:18 +0100 |
|---|---|---|
| committer | Ulrich Müller <ulm@gentoo.org> | 2019-11-26 20:23:19 +0100 |
| commit | d2cb9490dbee48a32f196d1aa80d7356a99d9fd8 (patch) | |
| tree | fcb073c0e19c78a4a1f0fc67472a9e16c33c19c4 | |
| parent | profiles: Mask net-wireless/bcm43xx-fwcutter for removal (diff) | |
| download | gentoo-d2cb9490.tar.gz gentoo-d2cb9490.tar.bz2 gentoo-d2cb9490.zip | |
estack.eclass: Properly restore shopt options.
Calling "eshopts_push; eshopts_pop" makes Portage report a QA issue:
* QA Notice: Global shell options changed and were not restored while calling 'src_prepare'
This is caused by some side effect in bash, by which disabling
the "posix" option (even if it was already disabled before) in a
non-interactive shell also disables the "expand_aliases" option.
Work around the problem by always saving and restoring both "set -o"
and "shopt" option sets.
Also fix "estack_push -s" which should not execute shopt when called
without further parameters.
Closes: https://bugs.gentoo.org/662586
Signed-off-by: Ulrich Müller <ulm@gentoo.org>
| -rw-r--r-- | eclass/estack.eclass | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/eclass/estack.eclass b/eclass/estack.eclass index f548abf8c283..b0177bdb358e 100644 --- a/eclass/estack.eclass +++ b/eclass/estack.eclass @@ -1,4 +1,4 @@ -# Copyright 1999-2017 Gentoo Foundation +# Copyright 1999-2019 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 # @ECLASS: estack.eclass @@ -153,12 +153,13 @@ evar_pop() { # eshopts_pop # @CODE eshopts_push() { + # Save both "shopt" and "set -o" option sets, because otherwise + # restoring noglob would disable expand_aliases by side effect. #662586 + estack_push eshopts "$(shopt -p -o) $(shopt -p)" if [[ $1 == -[su] ]] ; then - estack_push eshopts "$(shopt -p)" - [[ $# -eq 0 ]] && return 0 + [[ $# -le 1 ]] && return 0 shopt "$@" || die "${FUNCNAME}: bad options to shopt: $*" else - estack_push eshopts "$(shopt -p -o)" [[ $# -eq 0 ]] && return 0 set "$@" || die "${FUNCNAME}: bad options to set: $*" fi |