Skip to content

git: include autotools as build dep#5718

Merged
boegel merged 4 commits intoeasybuilders:developfrom
smoors:git
Feb 24, 2018
Merged

git: include autotools as build dep#5718
boegel merged 4 commits intoeasybuilders:developfrom
smoors:git

Conversation

@smoors
Copy link
Copy Markdown
Contributor

@smoors smoors commented Jan 24, 2018

No description provided.

@smoors
Copy link
Copy Markdown
Contributor Author

smoors commented Feb 22, 2018

@wpoely86 review?

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 22, 2018

@smoors Looks good, I made this consistent across recent git versions via smoors#2 . That includes a sync with current develop, so it's impossible to review, but it basically boils down to the last commit 42b432c

@boegel boegel added this to the next release (3.5.2 or 3.6.0) milestone Feb 22, 2018
@boegel boegel added the bug fix label Feb 22, 2018
sync with develop & make adding of Autotools build dep consistent for recent git versions
@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 24, 2018

Test report by @boegel
SUCCESS
Build succeeded for 5 out of 5 (4 easyconfigs in this PR)
node2316.phanpy.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/32f6f15b7bff4eda2ca4ec2fbb0ef38d for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 24, 2018

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (4 easyconfigs in this PR)
node2111.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/05927bf58a810b9de63f2b24375086c5 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 24, 2018

Going in, thanks @smoors!

@boegel boegel merged commit 5ed49dd into easybuilders:develop Feb 24, 2018
@smoors smoors deleted the git branch January 27, 2020 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants