[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