diff options
| author | Sam James <sam@gentoo.org> | 2023-06-23 05:28:57 +0100 |
|---|---|---|
| committer | Sam James <sam@gentoo.org> | 2023-06-23 05:33:00 +0100 |
| commit | cb8b981252e741096e9bf492471ba6fc6430b852 (patch) | |
| tree | 84dbf94a02339f987ebf8bf60c2cd4604bc59b44 /app-shells/bash/bash-9999.ebuild | |
| parent | 0386904bba5c9d824eec5348f5fc24901413bd7a (diff) | |
| download | gentoo-cb8b981252e741096e9bf492471ba6fc6430b852.tar.gz gentoo-cb8b981252e741096e9bf492471ba6fc6430b852.tar.bz2 gentoo-cb8b981252e741096e9bf492471ba6fc6430b852.zip | |
app-shells/bash: require Bison as yacc implementation
Upstream only test with Bison and require GNUisms like YYEOF and
YYERRCODE. The former at least may be in POSIX soon:
https://www.austingroupbugs.net/view.php?id=1269.
configure warns on use of non-Bison but doesn't abort. The result
may misbehave at runtime.
Noticed with recently added bash-5.2_p15-shell-parser-reset-issue.patch
(which is blameless in itself).
A simple test with a broken Bash is:
```
$ /var/tmp/portage/app-shells/bash-5.2_p15-r4/image/bin/bash -n /lib/gentoo/functions.sh
/lib/gentoo/functions.sh: line 104: syntax error near unexpected token `}'
/lib/gentoo/functions.sh: line 104: `}'
```
Reference: 3ee2d707a299f352b6970af459b0c25c356cbb25
Reference: dde3a81f420e745fe884b6535796129192f02561
Signed-off-by: Sam James <sam@gentoo.org>
Diffstat (limited to 'app-shells/bash/bash-9999.ebuild')
| -rw-r--r-- | app-shells/bash/bash-9999.ebuild | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/app-shells/bash/bash-9999.ebuild b/app-shells/bash/bash-9999.ebuild index af8066bbfd29..eeff3e0b3023 100644 --- a/app-shells/bash/bash-9999.ebuild +++ b/app-shells/bash/bash-9999.ebuild @@ -98,9 +98,9 @@ fi RDEPEND=" ${DEPEND} " -# We only need yacc when the .y files get patched (bash42-005, bash51-011) +# We only need bison (yacc) when the .y files get patched (bash42-005, bash51-011) BDEPEND=" - app-alternatives/yacc + sys-devel/bison pgo? ( dev-util/gperf ) verify-sig? ( sec-keys/openpgp-keys-chetramey ) " @@ -178,6 +178,13 @@ src_prepare() { } src_configure() { + # Upstream only test with Bison and require GNUisms like YYEOF and + # YYERRCODE. The former at least may be in POSIX soon: + # https://www.austingroupbugs.net/view.php?id=1269. + # configure warns on use of non-Bison but doesn't abort. The result + # may misbehave at runtime. + unset YACC + local myconf=( --disable-profiling |