-
Notifications
You must be signed in to change notification settings - Fork 699
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
{devel}[GCCcore/6.4.0] Qt5 5.10.1 + deps #6335
Conversation
Test report by @geimer |
Test report by @boegel |
Test report by @boegel |
Test report by @verdurin |
Test report by @verdurin |
@verdurin broken test report was because |
@boegel yes, I have a little menagerie of nodes across the UK for these tests, not all of which have lots of RAM... |
] | ||
|
||
dependencies = [ | ||
('Python', '2.7.14', '-bare'), |
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.
Before merging this, I'd like to settle the discussion on #5072
Thanks for clarifying @verdurin |
version = '39.1.0' | ||
versionsuffix = '-Python-%(pyver)s-bare' | ||
|
||
homepage = 'http://pypi.python.org/pypi/setuptools/' |
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.
@geimer this should be reverted to https
, unless there's a good reason for the change.
All but setuptools are codes that probably would benefit quite a lot from being compiled by Intel instead of GCC. So from my point of view they should be moved up a step in the toolchain. I.e., to GCC (or higher) instead of GCCcore, and then have an intel version of them too. And Qt5 5.10.1 needs, according to #7086, a gperf dependency. |
With #8201 being merged, this PR is now obsolete. |
Thanks for following up on this and closing the PR @geimer! |
Factored out from #6328