Skip to content

fix broken installation for Python 3.6.2 & 3.6.3 with PyNaCl as dep for paramiko extension by explicitely including previous PyNaCl version as extension#6971

Merged
migueldiascosta merged 1 commit intoeasybuilders:developfrom
boegel:py36_PyNaCl
Oct 7, 2018
Merged

fix broken installation for Python 3.6.2 & 3.6.3 with PyNaCl as dep for paramiko extension by explicitely including previous PyNaCl version as extension#6971
migueldiascosta merged 1 commit intoeasybuilders:developfrom
boegel:py36_PyNaCl

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 6, 2018

The installation of Python 3.6.2 & 3.6.3 with intel toolchain is broken because of a new PyNaCl release (1.3.0) that gets pulled in via paramiko which is listed as an extension, see failing test reports in #6946

This wasn't an issue before when the previous PyNaCl release was being pulled in, and as such it's a clear example of why we should start enforcing download_dep_fail = True in Python easyconfigs.

PyNaCl is already listed as extension in newer Python 3.x easyconfigs, and for Python 3.6.1 & older an older version of paramiko is included where the problem doesn't pop up

…or paramiko extension by explicitely including previous PyNaCl version as extension
@boegel boegel added the bug fix label Oct 6, 2018
@boegel boegel added this to the 3.7.1 milestone Oct 6, 2018
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 6, 2018

Test report by @boegel
SUCCESS
Build succeeded for 7 out of 7 (7 easyconfigs in this PR)
node2445.golett.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/d76c721c27c8c1dc13002c1ef51a0f26 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 6, 2018

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

Copy link
Copy Markdown
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Copy Markdown
Member

Going in, thanks @boegel!

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