Skip to content

Fix incorrect order of extensions for Python-2.7.14 easyconfigs. pycparser and cffi must come before cryptography.#8495

Merged
boegel merged 2 commits intoeasybuilders:developfrom
akesandgren:20190615192030_new_pr_Python2714
Jun 21, 2019
Merged

Fix incorrect order of extensions for Python-2.7.14 easyconfigs. pycparser and cffi must come before cryptography.#8495
boegel merged 2 commits intoeasybuilders:developfrom
akesandgren:20190615192030_new_pr_Python2714

Conversation

@akesandgren
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm, test report coming up...

@boegel boegel added this to the next release (3.9.3) milestone Jun 20, 2019
@boegel boegel added the bug fix label Jun 20, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 20, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 21, 2019

Going in, thanks @akesandgren!

@boegel boegel merged commit 473b208 into easybuilders:develop Jun 21, 2019
@akesandgren akesandgren deleted the 20190615192030_new_pr_Python2714 branch June 21, 2019 07:22
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 21, 2019

Test report by @boegel
SUCCESS
Build succeeded for 9 out of 9 (9 easyconfigs in this PR)
node3104.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/83c04bff198872b538fcc8082d84fc77 for a full test report.

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