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

[MDEPLOY-291] Update POM parent and Maven #22

Merged
merged 4 commits into from May 6, 2022

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Apr 22, 2022

Update plugin build:

  • update parent to 36
  • update maven to 3.2.5
  • leave the plugin to Java7
  • warn user if older that Maven 3.9.0 is used to deploy a maven-plugin

This plugin along with m-install-p versions 3.x line (3.0.0-Mx) does not work with any so far released Maven 3.x line (see MNG-7055). The ONLY maven 3.x they will be usable with is upcoming 3.9.0, that has fix for MNG-7055 backported, but OTOH, it is already Java8, so no point in keeping the plugin on anything below. This all stands, but ONLY for maven-plugin packaging.


https://issues.apache.org/jira/browse/MDEPLOY-291

Update plugin build:
* update parent to 36
* update maven to 3.8.5 (reasoning below)
* up the plugin to Java8

This plugin along with m-install-p versions 3.x line (3.0.0-Mx) does not work with any so far released Maven 3.x line (see MNG-7055). The ONLY maven 3.x they will be usable with is upcoming 3.9.0, that has fix for MNG-7055 backported, but OTOH, it is already Java8, so no point in keeping the plugin on anything below.

---

https://issues.apache.org/jira/browse/MDEPLOY-291
pom.xml Outdated Show resolved Hide resolved
* leave project java7
* up to maven 3.2.5 (not 3.8.5)
* warn if maven-plugin is deployed with maven older than 3.9.0
@cstamas cstamas requested a review from michael-o May 2, 2022 15:34
<dependency>
<groupId>org.eclipse.aether</groupId>
<artifactId>aether-api</artifactId>
<version>${resolverVersion}</version>
Copy link
Member

Choose a reason for hiding this comment

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

AFAIK, this one is provided https://maven.apache.org/ref/3.8.5/maven-core/core-extensions.html
(notice: but not aether-util)

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed, but to keep them paired, as here both are used by the plugin, I set both to compile (GenericVersion spans both)

@cstamas
Copy link
Member Author

cstamas commented May 6, 2022

@michael-o let's push this along with m-install-p and make 3.0.0 release of both soon, it is really time for it 😄

Copy link
Member

@slachiewicz slachiewicz left a comment

Choose a reason for hiding this comment

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

So this plugin must wait for 3.9?

@cstamas
Copy link
Member Author

cstamas commented May 6, 2022

So this plugin must wait for 3.9?

No, ignore my original PR comment (is edited exactly to reflect that). Initially I did create this PR to up expectation to 3.9.x but later we discussed: the issue is ONLY about packaging maven-plugin AND if older maven than 3.9.0 used. Hence, if you review PR you will see there a big fat warning about it that is triggered when these two conditions are met. For everything else, even today's Maven is okay to be used with.

So this PR is plain "Update Maven to 3.2.5 and parent POM..." with this extra twist.

@slachiewicz slachiewicz self-requested a review May 6, 2022 07:37
@cstamas cstamas merged commit 6eb066e into master May 6, 2022
@cstamas cstamas deleted the MDEPLOY-291-update-plugin branch May 6, 2022 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants