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

Uprade to Apache Maven 3.8.6 #4333

Merged
merged 1 commit into from Jul 12, 2022
Merged

Uprade to Apache Maven 3.8.6 #4333

merged 1 commit into from Jul 12, 2022

Conversation

ebarboni
Copy link
Contributor

@ebarboni ebarboni commented Jul 5, 2022

This is to upgrade embedded maven to 3.8.6 but overlay libs are incorrect

@mbien mbien added Upgrade Library Library (Dependency) Upgrade Maven [ci] enable "build tools" tests labels Jul 5, 2022
@mbien mbien added this to the NB15 milestone Jul 5, 2022
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.

Looks sane to me. A quick test with a freshly build NetBeans let me download dependencies, got artifact details, got sources and it build a project.

@mbien
Copy link
Member

mbien commented Jul 9, 2022

but overlay libs are incorrect

what does this mean?

Also I believe I linked the wrong issue. This is embedded, not the bundled maven, right?

@matthiasblaesing
Copy link
Contributor

We carry only one maven version, and this is the update - this is a screenshot from the build (see version output):

image

Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

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

lets get this in :)

@neilcsmith-net
Copy link
Member

Looks good.

but overlay libs are incorrect

As in ignored-overlaps? In what way incorrect? Worth correcting?

This is embedded, not the bundled maven, right?

Yes, maven.embedder is the module that bundles (or embeds) Maven. Somewhat confusing collision of terminology! 😄

@mbien
Copy link
Member

mbien commented Jul 11, 2022

Yes, maven.embedder is the module that bundles (or embeds) Maven. Somewhat confusing collision of terminology! smile

all good, just wanted to be sure that i linked the right issue :)

@ebarboni
Copy link
Contributor Author

unsure why commit-validation fail

@mbien
Copy link
Member

mbien commented Jul 12, 2022

unsure why commit-validation fail

the ValidateLayer tests are a bit broken at the moment #4211 #4218

@ebarboni
Copy link
Contributor Author

so I merge :D

@mbien
Copy link
Member

mbien commented Jul 12, 2022

so I merge :D

I don't know. Probably? Depends on what

but overlay libs are incorrect

means :)

@ebarboni
Copy link
Contributor Author

well in fact the overlay was looking bad because ant clean were not removing the jars. on a fresh clone seems ok.

@mbien
Copy link
Member

mbien commented Jul 12, 2022

mystery solved. lets merge :D

@ebarboni ebarboni merged commit 9783a43 into apache:master Jul 12, 2022
@ebarboni ebarboni deleted the maven-3.8.6 branch July 15, 2022 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maven [ci] enable "build tools" tests Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bundled Maven 3.8.5 can't resolve transitive dependencies on sub-projects (modules).
4 participants