Skip to content

8300805: Update autoconf build-aux files with latest from 2022-09-17#3206

Open
gnu-andrew wants to merge 1 commit into
openjdk:masterfrom
gnu-andrew:autoconf-update
Open

8300805: Update autoconf build-aux files with latest from 2022-09-17#3206
gnu-andrew wants to merge 1 commit into
openjdk:masterfrom
gnu-andrew:autoconf-update

Conversation

@gnu-andrew

@gnu-andrew gnu-andrew commented May 27, 2026

Copy link
Copy Markdown
Member

I came across this patch when trying to work out how to backport JDK-8285630 to fix the RISC-V Zero build. That fix is only available in 17u as part of a huge patch to bring in the RISC-V JIT port. It turns out that this autoconf update supersedes both that change and JDK-8265192, an additional MacOS on AArch64 fix, by updating the over a decade old autoconf files to new versions which natively support architectures like AArch64, RISC-V and LoongArch which the JDK has had to add overrides for.

The backport is not clean because of those two earlier fixes being absent in 11u and the lack of Windows WSL support added by JDK-8215445 and JDK-8257679



Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • JDK-8285630 needs maintainer approval
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • JDK-8265192 needs maintainer approval
  • JDK-8300805 needs maintainer approval

Issues

  • JDK-8300805: Update autoconf build-aux files with latest from 2022-09-17 (Enhancement - P3)
  • JDK-8285630: Fix a configure error in RISC-V cross build (Bug - P4)
  • JDK-8265192: [macos_aarch64] configure script fails if GNU uname in PATH (Bug - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk11u-dev.git pull/3206/head:pull/3206
$ git checkout pull/3206

Update a local copy of the PR:
$ git checkout pull/3206
$ git pull https://git.openjdk.org/jdk11u-dev.git pull/3206/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 3206

View PR using the GUI difftool:
$ git pr show -t 3206

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk11u-dev/pull/3206.diff

Using Webrev

Link to Webrev Comment

@bridgekeeper

bridgekeeper Bot commented May 27, 2026

Copy link
Copy Markdown

👋 Welcome back andrew! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk

openjdk Bot commented May 27, 2026

Copy link
Copy Markdown

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk openjdk Bot changed the title Backport f279c751a5b2132c11aa3a31084ee37f1a349150 8300805: Update autoconf build-aux files with latest from 2022-09-17 May 27, 2026
@openjdk

openjdk Bot commented May 27, 2026

Copy link
Copy Markdown

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk Bot added the backport Port of a pull request already in a different code base label May 27, 2026
@gnu-andrew

Copy link
Copy Markdown
Member Author

/issue add 8285630,8265192

@openjdk

openjdk Bot commented May 27, 2026

Copy link
Copy Markdown

@gnu-andrew
Adding additional issue to issue list: 8285630: Fix a configure error in RISC-V cross build.

Adding additional issue to issue list: 8265192: [macos_aarch64] configure script fails if GNU uname in PATH.

@gnu-andrew gnu-andrew marked this pull request as ready for review May 27, 2026 05:55
@openjdk openjdk Bot added the rfr Pull request is ready for review label May 27, 2026
@mlbridge

mlbridge Bot commented May 27, 2026

Copy link
Copy Markdown

Webrevs

@vieiro

vieiro commented May 27, 2026

Copy link
Copy Markdown
Contributor

FWIW, this builds fine in rhel-8.10 / gcc 8.5.0 / autoconf 2.69.

@gnu-andrew

Copy link
Copy Markdown
Member Author

/issue add 8285630,8265192

FWIW, this builds fine in rhel-8.10 / gcc 8.5.0 / autoconf 2.69.

Thanks. I appreciate the testing.

@openjdk

openjdk Bot commented May 27, 2026

Copy link
Copy Markdown

@gnu-andrew
Updating description of additional solved issue: 8285630: Fix a configure error in RISC-V cross build.

Updating description of additional solved issue: 8265192: [macos_aarch64] configure script fails if GNU uname in PATH.

@zzambers

zzambers commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

I have tested risc-v cross-build of this PR (x86_64 debian trixie build OS + risc-v debian trixie sysroot + temurin boot jdk). Configured with:
bash configure --with-jvm-variants=zero --openjdk-target=riscv64-linux-gnu --with-sysroot=$SYSROOT --with-boot-jdk=$BOOT_JDK --with-zlib=bundled --enable-debug

Build finished successfully. Resulting binary works on risc-v board (also debian trixie, done just --version test).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

3 participants