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.

Sign up for GitHub

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

Jump to bottom

consistently fix source_urls for numactl v2.0.11 #5296

Merged
merged 1 commit into from Nov 3, 2017
Merged

consistently fix source_urls for numactl v2.0.11 #5296

merged 1 commit into from Nov 3, 2017

Conversation

boegel
Copy link
Member

@boegel boegel commented Nov 3, 2017

cfr. #5285 and #5286

cc @verdurin, @vanzod

@boegel boegel added the bug fix label Nov 3, 2017
@boegel boegel added this to the 3.5.0 milestone Nov 3, 2017
@boegel boegel mentioned this pull request Nov 3, 2017
Closed
verdurin
verdurin approved these changes Nov 3, 2017
View reviewed changes
Copy link
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks fine - I can't run test builds of all these though.

@boegel
Copy link
Member Author

boegel commented Nov 3, 2017

Test report by @boegel
SUCCESS
Build succeeded for 20 out of 20 (20 easyconfigs in this PR)
node2130.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/4bc45c07b1cd811f99ab1e0f568dcb50 for a full test report.

@verdurin
Copy link
Member

verdurin commented Nov 3, 2017

Going in, thanks @boegel!

@verdurin verdurin merged commit 781e990 into easybuilders:develop Nov 3, 2017
@boegel boegel deleted the numactl2011_fix_source_urls branch November 3, 2017 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@verdurin verdurin verdurin approved these changes

Assignees
No one assigned
Labels
bug fix
Projects
None yet
Milestone
3.5.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@boegel @verdurin