Skip to content

Use the recommended directory layout for native-image metadata#12951

Merged
normanmaurer merged 1 commit into
netty:4.1from
gradinac:gradinac/fix-nbt-metadata-path-warnings
Nov 2, 2022
Merged

Use the recommended directory layout for native-image metadata#12951
normanmaurer merged 1 commit into
netty:4.1from
gradinac:gradinac/fix-nbt-metadata-path-warnings

Conversation

@gradinac

@gradinac gradinac commented Nov 1, 2022

Copy link
Copy Markdown
Contributor

Motivation:

The recommended directory layout for native-image metadata is META-INF/native-image/<groupId>/<artifactId>/.... Recent Native Build Tools versions will print a warning if a native-image.properties file is detected and it's path doesn't adhere to the above layout.

Modification:

This PR moves all native-image related metadata so that it conforms to the above layout.
This PR also ensures the above layout is used for the automatically generated handler metadata. The new layout of this metadata is META-INF/native-image/<groupId>/<artifactId>/automatic/handlers.

Result:

Fixes #12934

I'll also create a Netty 5 PR with the similar changes

@gradinac

gradinac commented Nov 1, 2022

Copy link
Copy Markdown
Contributor Author

Looking at the naming of the generated handler metadata folder, I think META-INF/native-image/<groupId>/<artifactId>/generated/handlers would be a better name than META-INF/native-image/<groupId>/<artifactId>/automatic/handlers - I'll push a commit that changes this

…ify automatic handler metadata generator usage.
@normanmaurer normanmaurer added this to the 4.1.85.Final milestone Nov 2, 2022
@normanmaurer normanmaurer merged commit fe0b5ce into netty:4.1 Nov 2, 2022
@normanmaurer

Copy link
Copy Markdown
Member

Thanks a lot @gradinac

@gradinac

gradinac commented Nov 2, 2022

Copy link
Copy Markdown
Contributor Author

Thank you very much for looking into this PR :)

normanmaurer pushed a commit that referenced this pull request Nov 8, 2022
…ify automatic handler metadata generator usage. (#12952)

Motivation and Modification:

This PR modifies the native-image metadata directory layout in exactly the same way as #12951 does for Netty 4.

Relates to #12934
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.

Maybe the GraalVM reachability metadata generated by Netty 4.1 is placed in the wrong path

2 participants