-
Notifications
You must be signed in to change notification settings - Fork 746
{lang}[foss/2018b,intel/2018b] Python v3.7.0 #6480
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.
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
{lang}[foss/2018b,intel/2018b] Python v3.7.0 #6480
Conversation
eda2271
to
a4c919b
Compare
Test report by @boegel |
Test report by @boegel |
Test report by @vanzod |
Test report by @boegel |
@vanzod Actual problem isn't caught in the partial log included in the test report, please provide a full (debug) log? |
Test report by @wpoely86 |
Test report by @migueldiascosta |
CentOS 6.x system OpenSSL is too old for python 3.7.0:
which then leads to https download errors (see test report) Maybe update the OpenSSL dependency comment so that people are aware? |
@migueldiascosta How specifically would you update that comment? Also, I'm not sure if that's going to make much of a difference, do you think people will look at comments in the easyconfig when try run into a build failure like that for Python 3.7.0? |
@boegel Here is the full debug log: https://gist.github.com/e1a7f1c5c417d559764fdae547ad3a31 It seems a problem with related to building |
@vanzod I think this is the culprit: So maybe including (the one about |
08ffe7c
to
afdfc02
Compare
@vanzod Extending |
Test report by @boegel |
Test report by @boegel |
Test report by @boegel |
Test report by @verdurin |
After noticing that there was an update for As it turns out, this makes the installation of |
Test report by @verdurin |
@boegel w.r.t. to the OpenSSL comment, something like "Python 3.7 requires OpenSSL >= 1.0.2, uncomment dependency if OS version is too old"? with #6567, it works for me (regarding people not reading the comment anyway, would it be feasible to enhance in any case, I don't think any of this should block merging this PR |
961d828
to
a33e25c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Test report by @boegel |
Test report by @bartoldeman |
Test report by @bartoldeman Note: this last test report had the easyconfig modified to not comment out the OpenSSL dependency. |
Test report by @boegel |
Test report by @boegel |
Test report by @vanzod |
Ok. Looks good to me. Plenty of test reports, thanks everyone. @boegel proposed merging, I will do it. |
(created using
eb --new-pr
)depends on
#6473(ncurses),#6474(bzip2),#6479(GMP, SQLite, ...),#6409(intel/2018b
),#6424(foss/2018b
)