Skip to content

Commit

Permalink
toolchain.eclass: never pass --enable-libvtv to ./configure
Browse files Browse the repository at this point in the history
gcc treats --enable-libvtv as an override on top of
autodetection. It it never what we want. Happens to break
at least powerpc64 cross-compilers:
    https://bugs.gentoo.org/661252

Reported-by: Ilia Mirkin
Closes: https://bugs.gentoo.org/661252
Signed-off-by: Sergei Trofimovich <[email protected]>
  • Loading branch information
Sergei Trofimovich committed Feb 15, 2019
1 parent 615c99f commit f437c7c
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions eclass/toolchain.eclass
Original file line number Diff line number Diff line change
Expand Up @@ -1278,7 +1278,8 @@ toolchain_src_configure() {
if in_iuse vtv ; then
confgcc+=(
$(use_enable vtv vtable-verify)
$(use_enable vtv libvtv)
# See Note [implicitly enabled flags]
$(usex vtv '' --disable-libvtv)
)
fi

Expand Down Expand Up @@ -2514,7 +2515,9 @@ toolchain_death_notice() {
#
# GCC does not follow this pattern and instead overrides autodetection
# of the feature and enables it unconditionally.
# See https://gcc.gnu.org/PR85663
# See bugs:
# https://gcc.gnu.org/PR85663 (libsanitizer on mips)
# https://bugs.gentoo.org/661252 (libvtv on powerpc64)
#
# Thus safer way to enable/disable the feature is to rely on implicit
# enabled-by-default state:
Expand Down

0 comments on commit f437c7c

Please sign in to comment.