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

Upgrade to Apache Maven 3.8.3 #3239

Closed
wants to merge 1 commit into from
Closed

Conversation

ebarboni
Copy link
Contributor

Hi, upgrading 3.8.3 in the maven.embedder. Seems very fast on reactor build compare to previous.

I hope I catch all the file to license and hash.

@ebarboni ebarboni added the Upgrade Library Library (Dependency) Upgrade label Oct 12, 2021
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

In general this looks good to me. RAT looks good and eyeballing the maven-embedded zip and looking at the license changes this looks good. I left two inline comments.

I assume this was tested with real projects and they still work correctly.

java/maven.embedder/external/binariesembedded-list Outdated Show resolved Hide resolved
java/maven.embedder/external/binariesembedded-list Outdated Show resolved Hide resolved
@ebarboni
Copy link
Contributor Author

It works for my limited point of view.

@matthiasblaesing
Copy link
Contributor

matthiasblaesing commented Oct 14, 2021

The update breaks listing dependencies in at least one project for me. This is a "play ground" project and hast a big chunk of dependencies. With the updated maven-embedder the dependencies are not listed anymore.

From a quick look at the netbeans code it looks as if the MavenProject#getArtifacts method changed its beavior and returns an empty list.

@ebarboni ebarboni added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Oct 20, 2021
@ebarboni
Copy link
Contributor Author

@matthiasblaesing I'm not sure how it happens but I first was able to see dependencies and after a while dependencies folder became empty.

@neilcsmith-net
Copy link
Member

From a quick look at the netbeans code it looks as if the MavenProject#getArtifacts method changed its beavior and returns an empty list.

So possibly this? https://issues.apache.org/jira/browse/MNG-7285

Perhaps check if it can be reproduced with 3.8.1?

@ebarboni
Copy link
Contributor Author

maybe better to wait for 3.8.4 :D. I just see the speed gain using 3.8.3, maybe would have help default speed but not seeing dependency is worst.

@neilcsmith-net
Copy link
Member

Just a note that a fix (actually revert) seems to have been merged into maven 3.8.x branch now, so keep an eye out for 3.8.4 and hopefully this will be OK. Us and Eclipse Maven support waiting on this one! 😄

@ebarboni
Copy link
Contributor Author

closing waiting for next maven version

@ebarboni ebarboni closed this Nov 15, 2021
@ebarboni ebarboni deleted the maven383 branch July 15, 2022 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Don't merge this PR, it is not ready or just demonstration purposes. Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants