RFR: 8333128: Linux x86_32 configure fail with --with-hsdis=binutils --with-binutils-src
Julian Waters
jwaters at openjdk.org
Mon Jun 3 09:02:07 UTC 2024
On Sun, 2 Jun 2024 02:49:47 GMT, SendaoYan <syan at openjdk.org> wrote:
> Hi all,
> This PR try to fix linux x86_32 configure fail with `--with-hsdis=binutils --with-binutils-src`. The libiberty.a locates in `build/linux-x86-server-fastdebug/configure-support/binutils-install/lib32` on linux ubuntu20. The change has been verified, the risk is low.
>
> Additional testing:
>
> - [x] linux x86_32 centos7 configure
> - [x] linux x86_64 centos7 configure
> - [x] linux x86_32 ubuntu20 configure
> - [x] linux x86_64 ubuntu20 configure
>
>
> [configure-linux-centos7-x86_32.log](https://github.com/user-attachments/files/15523974/configure-linux-centos7-x86_32.log)
> [configure-linux-centos7-x86_64.log](https://github.com/user-attachments/files/15523976/configure-linux-centos7-x86_64.log)
> [configure-ubuntu20-x86_32.log](https://github.com/user-attachments/files/15523977/configure-ubuntu20-x86_32.log)
> [configure-ubuntu20-x86_64.log](https://github.com/user-attachments/files/15523978/configure-ubuntu20-x86_64.log)
Don't forget about me! :)
make/autoconf/lib-hsdis.m4 line 274:
> 272: HSDIS_CFLAGS="-DLIBARCH_$OPENJDK_TARGET_CPU_LEGACY_LIB -I$BINUTILS_INSTALL_DIR/include"
> 273:
> 274: # libiberty ignores --libdir and may be installed in $BINUTILS_INSTALL_DIR/lib, $BINUTILS_INSTALL_DIR/lib32 or $BINUTILS_INSTALL_DIR/lib64
Nit: "or $BINUTILS_INSTALL_DIR/lib64" should probably be on the new line in this comment
-------------
Marked as reviewed by jwaters (Committer).
PR Review: https://git.openjdk.org/jdk/pull/19511#pullrequestreview-2093236451
PR Review Comment: https://git.openjdk.org/jdk/pull/19511#discussion_r1624067867
More information about the build-dev
mailing list