Skip to content

[MWAR-444] Update plugin (requires Maven 3.2.5+) - #20

Merged
dennisl merged 1 commit into
masterfrom
update-plugin
Apr 25, 2022
Merged

dennisl merged 1 commit into
masterfrom
update-plugin

Conversation

@cstamas

@cstamas cstamas commented Oct 11, 2021 •

Copy link
Copy Markdown
Member

Move maven to proper scope, update dependencies,
remove unused ones.


https://issues.apache.org/jira/browse/MWAR-444

Move maven to proper scope, update dependencies,
remove unused ones.
Comment thread pom.xml
<mavenArchiverVersion>3.5.1</mavenArchiverVersion>
<mavenFilteringVersion>3.1.1</mavenFilteringVersion>
<mavenVersion>3.1.0</mavenVersion>
<mavenVersion>3.2.5</mavenVersion>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this required? Personally I'm OK with it. However mostly I think we've been setting 3.1.1 as out minimum lately.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is requirement dictated by maven-plugin-testing-harness v 3.3.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we can stay with p-p-testing 3.1.0 as for the other 3.1.1. based plugins. Unfortunately, if the resources plugin will go for 3.2 then 3.2 will be our minimal version for everything (no other 3.x rel if we want to remove on 2.x plugins)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Personally I'm OK with that. In fact, I'd be willing to go at least as far as 3.3 as a minimum. Is it worth asking on the dev list to see if we can get consensus on this as a cross-project minimum from now on?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, agree. i have same opinion here

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I even have another opinion. When Maven 3.9.0 will be released by end of this year I would base ALL baseline to 3.5.4 and burry anything below. There is no reason to support 8-year old Maven. Maven 3.5.4 will be 3,5 years old by then.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I feel like there's a package manager I've encountered in the recent past (brew? Debian apt-get?) that installs 3.3.x by default. That's why I chose that version.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I feel like there's a package manager I've encountered in the recent past (brew? Debian apt-get?) that installs 3.3.x by default. That's why I chose that version.

That is purely a Debian problem. Moreover, Debien screws up Maven to FUBAR.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

brew on Mac OS install the latest version of Maven

@cstamas cstamas self-assigned this Oct 13, 2021
@cstamas
cstamas marked this pull request as draft October 13, 2021 20:53
@cstamas cstamas changed the title Update plugin [MWAR-444] Update plugin (requires Maven 3.2.5+) Oct 15, 2021
@cstamas
cstamas marked this pull request as ready for review October 15, 2021 12:12
Comment thread pom.xml
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-archiver</artifactId>
<version>4.2.2</version>
<version>4.2.5</version>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

4.2.7 is out

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

4.2.7 is out

But until the maven-war-plugin uses Java 8, that would require to downgrade commons-compress and plexus-utils, and my guess is that to simplify transitive dependency management, this just sets the latest that has bytecode 51.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We are moving everything to 8

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sure, yet this project is still Java 7:

<javaVersion>7</javaVersion>

So, ideally bump here to Java 8 and update plexus-archiver to 4.2.7, or bump it on another PR.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PR opened #23

@dennisl
dennisl merged commit 35cec22 into master Apr 25, 2022
dennisl added a commit to mdh-se/maven-war-plugin that referenced this pull request Apr 25, 2022
Merge pull request apache#20 from apache/update-plugin
@asfgit
asfgit deleted the update-plugin branch June 2, 2022 11:40
@jira-importer

Copy link
Copy Markdown

Resolve #516

1 similar comment
@jira-importer

Copy link
Copy Markdown

Resolve #516

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.

8 participants