Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Upgrade Qt to 5.6 #14102

Closed
3 tasks done
vitallium opened this issue Mar 16, 2016 · 21 comments
Closed
3 tasks done

Upgrade Qt to 5.6 #14102

vitallium opened this issue Mar 16, 2016 · 21 comments
Assignees
Labels

Comments

@vitallium
Copy link
Collaborator

vitallium commented Mar 16, 2016

Qt 5.6 was released today http://blog.qt.io/blog/2016/03/16/qt-5-6-released/

Change log: https://wiki.qt.io/New_Features_in_Qt_5.6

This is a stub issue for referencing in future.

Check list:

  • Linux
  • OS X
  • Windows

Cheers!

@vitallium vitallium self-assigned this Mar 16, 2016
@vitallium vitallium added this to the Release 2.2 milestone Mar 16, 2016
vitallium added a commit to vitallium/qtbase that referenced this issue Mar 16, 2016
vitallium added a commit to vitallium/qtbase that referenced this issue Mar 16, 2016
@Vanuan
Copy link

Vanuan commented Mar 18, 2016

AFAIK Qt 5.6 removed QtWebkit support and migrated to chromium. Isn't that true?

@Vanuan
Copy link

Vanuan commented Mar 18, 2016

With Qt 5.6 the following modules are no longer part of the release packages, but users can still build them from source:

  • Qt WebKit

Qt WebEngine

  • Based on Chromium 45

@vitallium
Copy link
Collaborator Author

@Vanuan that's not a problem. Updated WebKit is coming - https://github.com/annulen/webkit

@Vanuan
Copy link

Vanuan commented Mar 18, 2016

Ah, so phantomjs would use WebKit directly?

@vitallium
Copy link
Collaborator Author

@Vanuan kind of. We will maintain our own QtWebkit.

@annulen
Copy link

annulen commented Mar 18, 2016

@Vanuan In fact there is a community release of QtWebKit 5.6:
http://download.qt.io/community_releases/5.6/5.6.0/

@elachlan
Copy link

elachlan commented Apr 6, 2016

Am I correct in assuming this would fix the security issue that is blocking package acceptance in Debian?
https://security-tracker.debian.org/tracker/CVE-2013-4549

@annulen
Copy link

annulen commented Apr 6, 2016

QXmlSimpleReader is not used in QtWebKit and PhantomJS, so I believe this CVE has no relation to PhantomJS

@elachlan
Copy link

elachlan commented Apr 6, 2016

Does anyone know the process to have it removed from the CVE?

@zackw
Copy link
Collaborator

zackw commented Apr 6, 2016

@elachlan Until there's a maintained QtWebkit that is keeping up with Webkit security advisories and PhantomJS can use both that and unmodified system-provided Qt, packaging it for Debian - or any other distro - is inappropriate. I specifically told them a few months ago that they should drop the package altogether. Maybe I should try to prod that process along.

Until that time, also, it's probably unsafe to use PhantomJS on untrusted content, especially on Windows.

@vitallium
Copy link
Collaborator Author

Windows +

vitallium added a commit that referenced this issue May 18, 2016
Do a small cleanup also.

Issue: #14102
@vitallium
Copy link
Collaborator Author

Landed in master

vitallium added a commit that referenced this issue May 18, 2016
Do a small cleanup also.

Issue: #14102
vitallium added a commit that referenced this issue May 19, 2016
Do a small cleanup also.

Issue: #14102
@astefanutti
Copy link
Contributor

@vitallium, I've just tried compiling @5d99f2a on Mac OS X (10.11.5) and Qt WebKit build fails with the following error message:

rm -f /Users/astefanu/Development/phantomjs-test/src/qt/qtbase/lib/libQt5WebKit.a
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ar cq /Users/astefanu/Development/phantomjs-test/src/qt/qtbase/lib/libQt5WebKit.a 
ar: no archive members specified
usage:  ar -d [-TLsv] archive file ...
    ar -m [-TLsv] archive file ...
    ar -m [-abiTLsv] position archive file ...
    ar -p [-TLsv] archive [file ...]
    ar -q [-cTLsv] archive file ...
    ar -r [-cuTLsv] archive file ...
    ar -r [-abciuTLsv] position archive file ...
    ar -t [-TLsv] archive [file ...]
    ar -x [-ouTLsv] archive [file ...]
make[2]: *** [/Users/astefanu/Development/phantomjs-test/src/qt/qtbase/lib/libQt5WebKit.a] Error 1
make[1]: *** [sub-api-pri-make_first-ordered] Error 2
make: *** [sub-Source-QtWebKit-pro-make_first-ordered] Error 2

ERROR: Failed to build PhantomJS! Building Qt WebKit failed.

I've opened vitallium/qtwebkit#11 that fixes the issue.

ariya added a commit that referenced this issue May 26, 2016
Probably related to the new Qt (#14102)
@sowani
Copy link

sowani commented Feb 22, 2017

By any chance, https://github.com/annulen/webkit/releases/tag/qtwebkit-tp5 will be used in PhantomJS?

@vitallium
Copy link
Collaborator Author

vitallium commented Feb 22, 2017

@sowani by any chance - PhantomJS 2.5 already uses this fork ;-)

@phawxby
Copy link

phawxby commented Apr 25, 2017

@vitallium where is V2.5? I don't see it listed in releases?
https://github.com/ariya/phantomjs/releases

I'm suffering from issues with flexbox, blurring and broken webfonts (I believe this is Windows only) and I think this is due to the old Webkit version in V2.1
homepage--en-us--desktop

@annulen
Copy link

annulen commented Apr 25, 2017

@phawxby
Copy link

phawxby commented Apr 25, 2017

What on earth is it doing there?

@izzyonstage
Copy link

is there any tags to the source for the 2.5 build?
We have some custome PJS changes we make in-house and it would be good to be able to use a build that has tp5 in it.

@phawxby
Copy link

phawxby commented May 12, 2017

Medium/phantomjs#689

You can pull directly from my repo if you want although I would look at the pull conversation.
https://github.com/phawxby/phantomjs

@izzyonstage
Copy link

izzyonstage commented May 12, 2017

What I really need is the source code as I have some custom additions I need to add so the binaries are useless for me.

Also I am building for Windows platform as well as Linux

@ghost ghost removed the In progress label Dec 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

9 participants