Skip to content

{devel,vis}[foss/2018b] add missing gperf dep for Qt5 5.10.1 and new config for Qt5 5.11.2#7086

Merged
boegel merged 2 commits intoeasybuilders:developfrom
jhein32:qt5-in-foss2018b
Oct 31, 2018
Merged

{devel,vis}[foss/2018b] add missing gperf dep for Qt5 5.10.1 and new config for Qt5 5.11.2#7086
boegel merged 2 commits intoeasybuilders:developfrom
jhein32:qt5-in-foss2018b

Conversation

@jhein32
Copy link
Copy Markdown
Collaborator

@jhein32 jhein32 commented Oct 29, 2018

configs QT5.10.1 (bugfix) and QT5.11.2 (new) for foss/2018b

This replaces PR #7082, which got corrupted when attempting a merge conflict resolution.

configs QT5.10.1 (bugfix) and QT5.11.2 (new) for foss/2018b
akesandgren
akesandgren previously approved these changes Oct 29, 2018
Copy link
Copy Markdown
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

('libGLU', '9.0.0'),
('NSS', '3.39'),
('DBus', '1.13.6'),
('gperf', '3.1'),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@jhein32 We should apply this same fix in the other easyconfigs for Qt5 5.10.1?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I only had the issue in foss/2018b. I could build intel/2018a and foss/2018a without issues. Most likely I have an older version that doesn't force a webengine. gperf is only needed when the webengine is build - as noted in Slack, I had that failing and found a clear comment in the log: No gperf, no webengine.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ok, since you asked so nicely ;) I put a revised Qt 5.10.1 for foss/2018a.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I leave the intel/2018a config to be fixed once the webengine issues with intel have been fixed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@jhein32 I added gperf as a dep in #7010

@boegel boegel added this to the 3.8.0 milestone Oct 30, 2018
@boegel boegel changed the title {devel,vis}[foss/2018b] Bugfix in Qt5.10.1 and new config for Qt5.11.2 {devel,vis}[foss/2018b] add missing gperf dep for Qt5 5.10.1 and new config for Qt5 5.11.2 Oct 30, 2018
gperf added as a dep to the older Qt5 in foss/2018a (reviewer request)
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 30, 2018

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 30, 2018

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2606.swalot.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/8679dd92b09ec3939b203cca4539a3ff for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 30, 2018

Test report by @boegel
FAILED
Build succeeded for 1 out of 2 (2 easyconfigs in this PR)
node2002.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/abad0b1d60f42ddfedb156da2e08bb18 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 31, 2018

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 31, 2018

Going in, thanks @jhein32!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants