[riscv-port] Integrated: 8283020: riscv: Fix configure integration
Magnus Ihse Bursie
ihse at openjdk.java.net
Mon Mar 14 10:43:12 UTC 2022
On Fri, 11 Mar 2022 11:16:01 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> 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)
This pull request has now been integrated.
Changeset: 2a393bf5
Author: Magnus Ihse Bursie <ihse at openjdk.org>
Committer: Yadong Wang <yadongwang at openjdk.org>
URL: https://git.openjdk.java.net/riscv-port/commit/2a393bf5a591086d0558b4d3c9ac908997ddf916
Stats: 17 lines in 4 files changed: 9 ins; 5 del; 3 mod
8283020: riscv: Fix configure integration
Reviewed-by: fyang, fjiang
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/66
More information about the riscv-port-dev
mailing list