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

[MNG-7055] Fix G level metadata handling in 3.9.x #691

Merged
merged 4 commits into from
Apr 22, 2022

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Mar 13, 2022

Maven Artifact Transfer silently prevents group level metadata to reach Resolver and causes metadata loss on install/deploy.
Fix is to "bridge" this from maven-resolver-provider (and core) by reusing the actual metadata that
m-plugin-p:addPluginArtifactMetadata mojo adds, but m-a-t filters out.

This is backport of commit d141957 from master to maven-3.9.x branch.

Maven Artifact Transfer silently prevents group level metadata to reach Resolver and causes metadata loss on install/deploy.
Fix is to "bridge" this from maven-resolver-provider (and core) by reusing the actual metadata that
m-plugin-p:addPluginArtifactMetadata mojo adds, but m-a-t filters out.

This is backport of commit d141957 to maven-3.9.x branch.
@cstamas cstamas requested review from gnodet and michael-o March 13, 2022 18:08
@cstamas cstamas self-assigned this Mar 13, 2022
@cstamas
Copy link
Member Author

cstamas commented Mar 13, 2022

Was too eager, maybe lifecycle mappings should not be changed?

@cstamas
Copy link
Member Author

cstamas commented Mar 13, 2022

Why is CI building Java7?

@michael-o
Copy link
Member

Because I have forked 3.8.x which still had Java 7.

@cstamas
Copy link
Member Author

cstamas commented Mar 13, 2022

CI is still crap, one IT failed (unrelated), and even that failed as it could run clean goal??? (are jobs isolated or what is happening)

@cstamas
Copy link
Member Author

cstamas commented Apr 13, 2022

Verified locally the PR, all ok. @michael-o or @gnodet please review

@cstamas cstamas changed the title [MNG-7055] Fix G level metadata handling (#555) [MNG-7055] Fix G level metadata handling in 3.9.x (#555) Apr 13, 2022
@cstamas cstamas changed the title [MNG-7055] Fix G level metadata handling in 3.9.x (#555) [MNG-7055] Fix G level metadata handling in 3.9.x Apr 13, 2022
Copy link
Member

@michael-o michael-o left a comment

Choose a reason for hiding this comment

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

If there is an accompanying IT, please make sure that the required Maven version is altered also.

@cstamas cstamas merged commit 4c9dc5a into maven-3.9.x Apr 22, 2022
@cstamas cstamas deleted the MNG-7055-g-level-md-mvn-39x branch April 22, 2022 15:51
@gnodet gnodet mentioned this pull request May 24, 2023
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.

2 participants