Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the version and copyright date for libFFI #14835

Conversation

ChengJin01
Copy link

Signed-off-by: Cheng Jin jincheng@ca.ibm.com

@ChengJin01
Copy link
Author

Reviewer: @pshipton
FYI: @tajila

<h4>libFFI pre 3.4</h4>
libffi - Copyright (c) 1996-2021 Anthony Green, Red Hat, Inc and others.
<h4>libFFI 3.4.2</h4>
libffi - Copyright (c) 1996-2022 Anthony Green, Red Hat, Inc and others.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure 2022 is correct, 3.4.2 is released in 2021. Are we taking more recent versions than 3.4.2?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be ambiguous for version but we always take the latest changes from https://github.com/libffi/libffi instead of https://github.com/libffi/libffi/releases/tag/v3.4.2 (2021) which doesn't include the latest changes in RISC-V at #14699 which was merged earlier this year.

commit aa3fce08ba620c50db17215a9f14dd0f1facf741
Author: Andreas Schwab [schwab@linux-m68k.org](mailto:schwab@linux-m68k.org)
Date: Sun Feb 13 21:04:33 2022 +0100 <-------

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then I think we should use pre 3.5 as the version.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to pre 3.5 as suggested.

Signed-off-by: Cheng Jin <jincheng@ca.ibm.com>
@ChengJin01 ChengJin01 force-pushed the update_libffi_version_copyright_date branch from c04993f to f4aea17 Compare April 1, 2022 15:50
@pshipton pshipton merged commit da8311d into eclipse-openj9:master Apr 1, 2022
@pshipton
Copy link
Member

pshipton commented Apr 1, 2022

Can you pls backport "pre 3.4 & 2021" to the 0.31 and 0.32 branches. i.e. just the x86 changes are in those releases.

@ChengJin01
Copy link
Author

ChengJin01 commented Apr 1, 2022

PR with pre 3.4 & 2021 for 0.31 is created at #14837.

@ChengJin01
Copy link
Author

ChengJin01 commented Apr 1, 2022

PR with pre 3.4 & 2021 for 0.32 is created at #14838.

pshipton added a commit to pshipton/openj9 that referenced this pull request Dec 7, 2022
This should have been done with previous changes
eclipse-openj9#16305
eclipse-openj9#14835

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
pshipton added a commit to pshipton/openj9 that referenced this pull request Dec 7, 2022
This should have been done with previous changes
eclipse-openj9#16305
eclipse-openj9#14835

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants