[riscv-port] RFR: 8283020: riscv: Fix configure integration
Magnus Ihse Bursie
ihse at openjdk.java.net
Fri Mar 11 11:21:25 UTC 2022
The current implementation in the riscv-port modifies make/autoconf/build-aux/autoconf-* files. For legal reasons, this is not something that is allowed. Instead, we need to introduce patches in our wrapper scripts in the same directory.
This patch will restore the original contents of the autoconf-* files, and instead move this logic to the wrapper scripts.
This PR also fixes an indentation problem in configure.
I have not been able to verify this patch, since I do not have access to riscv hardware. I believe it is sufficient and correct, but it might be the case that config.sub also needs updating. I kindly request assistance in verifying this.
(Apart from these fixes, all build changes in the riscv-port looks good to me)
-------------
Commit messages:
- 8283020: riscv: Fix configure integration
Changes: https://git.openjdk.java.net/riscv-port/pull/66/files
Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=66&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8283020
Stats: 17 lines in 4 files changed: 9 ins; 5 del; 3 mod
Patch: https://git.openjdk.java.net/riscv-port/pull/66.diff
Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/66/head:pull/66
PR: https://git.openjdk.java.net/riscv-port/pull/66
More information about the riscv-port-dev
mailing list