Skip to content

Bump required Maven version to 3.2#660

Merged
asfgit merged 2 commits into
apache:masterfrom
swegner:maven-version
Jul 15, 2016
Merged

Bump required Maven version to 3.2#660
asfgit merged 2 commits into
apache:masterfrom
swegner:maven-version

Conversation

@swegner

@swegner swegner commented Jul 14, 2016

Copy link
Copy Markdown
Contributor

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Comment thread pom.xml
</developer>
</developers>

<prerequisites>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Prerequisites are not actually enforced in Maven 3, see: http://stackoverflow.com/a/18058572/33791

@swegner

swegner commented Jul 14, 2016

Copy link
Copy Markdown
Contributor Author

R: @kennknowles

@swegner swegner changed the title Bump required Maven version to 3.3 Bump required Maven version to 3.2 Jul 14, 2016
@kennknowles

Copy link
Copy Markdown
Member

I now remember what feature I really wanted, and it is available as of 3.2.1: Being able to explicitly exclude build of a project in the Reactor, since dependency exclusions don't work there.

@kennknowles

Copy link
Copy Markdown
Member

Another reference point is Ubuntu release history (other flavors also matter, of course). https://launchpad.net/ubuntu/+source/maven. Looks like a jump from 3.0 to 3.3 on 2015-08-05. So anyone using an OS that is still < 3.2 is probably used to having to install their own software, and it won't be too great a burden.

LGTM. Certainly we shouldn't have the incorrect version there. If there are any complaints, we can lower to 3.1.

@asfgit asfgit merged commit 254a71d into apache:master Jul 15, 2016
asfgit pushed a commit that referenced this pull request Jul 15, 2016
@swegner swegner deleted the maven-version branch August 5, 2016 23:16
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.

3 participants