Skip to content

enable building of QtWebEngine in Qt5 easyconfig using foss/2017b or intel/2017b#7302

Merged
boegel merged 7 commits intoeasybuilders:developfrom
vanzod:20181212155855_new_pr_Qt5580
Dec 14, 2020
Merged

enable building of QtWebEngine in Qt5 easyconfig using foss/2017b or intel/2017b#7302
boegel merged 7 commits intoeasybuilders:developfrom
vanzod:20181212155855_new_pr_Qt5580

Conversation

@vanzod
Copy link
Copy Markdown
Member

@vanzod vanzod commented Dec 12, 2018

(created using eb --new-pr)

@vanzod vanzod changed the title [foss/2017b,intel/2017b] Qt5 - Fix to allow build for QtWebEngine [foss/2017b,intel/2017b] Qt5 - Fix to allow build QtWebEngine - WIP Dec 13, 2018
@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Dec 13, 2018

In order to build QtWebEngine in Qt5/5.8.0 the libpci library is required from pciutils. I am now having hard times to let it see it...

@akesandgren
Copy link
Copy Markdown
Contributor

Kicking Travis due to changes in EasyBuild testing.

@akesandgren
Copy link
Copy Markdown
Contributor

Travis kicked

@akesandgren akesandgren reopened this Aug 30, 2019
@akesandgren
Copy link
Copy Markdown
Contributor

@vanzod Is this still to be considered WIP? Looks fairly similar to later Qt5 easyconfigs

@easybuilders easybuilders deleted a comment from boegelbot Dec 11, 2020
@boegel boegel changed the title [foss/2017b,intel/2017b] Qt5 - Fix to allow build QtWebEngine - WIP enable building of QtWebEngine in Qt5 easyconfig using foss/2017b or intel/2017b Dec 12, 2020
@boegel boegel added this to the 4.x milestone Dec 12, 2020
@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 13, 2020

@vanzod as part of the merge sprint I have reviewed this PR and made the necessary changes to allow its merge:

  • added the missing dependency on libpci, as already mentioned in previous comments
  • fixed the homepage URLs

I hope that you agree with these changes.
This PR now depends on #11871

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 13, 2020

Test report by @lexming
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node381.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/6c2d981052012ede426bdb00ca050d64 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 13, 2020

Test report by @lexming
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
node381.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/44c52292e64032fe57a842b8e44cef8f for a full test report.

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 13, 2020

One final change: Qt5-5.8.0-intel-2017b.eb cannot be built with WebEngine because it requires gcc:

Using gcc version ., but at least gcc version 4.7 is required to build Qt WebEngine.                                  
QtWebEngine will not be built.

@easybuilders easybuilders deleted a comment from boegelbot Dec 13, 2020
@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 13, 2020

Test report by @lexming
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node361.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/284b17232184bd827783f0791800339e for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Dec 14, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 14, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node2710.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/e3691c04084172d3ec9c1ae5a882f885 for a full test report.

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

@boegel boegel modified the milestones: 4.x, next release (4.3.3?) Dec 14, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 14, 2020

Going in, thanks @vanzod!

@boegel boegel merged commit 4b4e0fa into easybuilders:develop Dec 14, 2020
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.

4 participants