Skip to content

Conversation

@jormundur00
Copy link
Contributor

What does this PR do?

In this PR we add packaging of library-and-framework-list.json to all future graalvm-reachability-metadata releases. The data provided in this file can be used by Native Build Tools to improve the accuracy of the native image dynamic access analysis, and should hence be packaged with the future releases.

@jormundur00 jormundur00 requested a review from a team as a code owner October 28, 2025 10:52
@jormundur00 jormundur00 requested a review from melix October 28, 2025 10:52
Copy link
Member

@vjovanov vjovanov left a comment

Choose a reason for hiding this comment

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

LGTM, we need this in build tools to compute supported libraries.

Copy link
Member

@melix melix left a comment

Choose a reason for hiding this comment

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

Why did you have to add an if? Shouldn't the file always be present?

@jormundur00
Copy link
Contributor Author

Why did you have to add an if? Shouldn't the file always be present?

I added it just to be safe, so if someone decided to tinker with the repository and moved or removed the file, it wouldn't break the packaging. If you think this is not the way to go (and we SHOULD crash if someone decides to move/remove the file), I can remove the if.

@melix
Copy link
Member

melix commented Nov 4, 2025

I would remove the if. Nothing is worse than silent errors :)

@jormundur00 jormundur00 force-pushed the jvukicev/package-library-and-framework-list branch from d5d8c98 to 77e4186 Compare November 4, 2025 10:52
@vjovanov vjovanov merged commit 6e0358d into oracle:master Nov 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants