Skip to content

Maven#1094

Merged
boegel merged 5 commits intoeasybuilders:developfrom
JensTimmerman:maven
Sep 19, 2014
Merged

Maven#1094
boegel merged 5 commits intoeasybuilders:developfrom
JensTimmerman:maven

Conversation

@JensTimmerman
Copy link
Copy Markdown

@JensTimmerman JensTimmerman commented Sep 19, 2014

No description provided.

boegel
boegel reviewed Sep 19, 2014
View reviewed changes
Comment thread easybuild/easyconfigs/m/Maven/Maven-3.2.3.eb Outdated
Copy link
Copy Markdown
Member

@boegel boegel Sep 19, 2014

Choose a reason for hiding this comment

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

use %(version_major)s rather than hardcoding the 3

Copy link
Copy Markdown
Member

@boegel boegel Sep 19, 2014

Choose a reason for hiding this comment

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

use http://www.apache.org/dist/maven/maven-%(version_major)s/... instead

@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 19, 2014

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/8300a313f4e878cfa733 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 19, 2014

Looking good, going in. Thanks @JensTimmerman!

boegel added a commit that referenced this pull request Sep 19, 2014
@boegel
Merge pull request #1094 from JensTimmerman/maven
2cca7c3 
Maven
@boegel boegel merged commit 2cca7c3 into easybuilders:develop Sep 19, 2014
@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 28, 2014

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/e0bd846dbc68d20c7b87 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 28, 2014

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/93ae14d64cd9a8134a03 for a full test report.

@boegel boegel mentioned this pull request Oct 28, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JensTimmerman @boegel