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

2.0 #12329

Closed
wants to merge 2 commits into from
Closed

2.0 #12329

wants to merge 2 commits into from

Conversation

vitallium
Copy link
Collaborator

Long live PhantomJS 2.0!

What's inside:

  • moved to Qt 5 (5.3.0)
  • fresh version of Webkit (not the latest)
  • API are the same as in 1.9

@JamesMGreene
Copy link
Collaborator

🤘 You deserve much kudos, ;@vitallium, @KDAB, et al! Wish I could've helped more.

@ariya
Copy link
Owner

ariya commented Jun 24, 2014

Well done @vitallium and friends! I'll take a look at this shortly.

Quick question to all: should we just drop this into master? Of course, before that I can create 1.x branch to continue the current master (before it is being replaced by this one). What do you guys think @detro @JamesMGreene?

@vitallium
Copy link
Collaborator Author

I'd place it to a separate branch, to fix all things. For example, WebServer module works unstable.

@alexshk
Copy link

alexshk commented Jun 25, 2014

👍 Thanks a lot. This solved issue with OSX 10.10 Yosemite

@sveisvei
Copy link

Kudos! What version of webkit will most likely ship with 2.0?

@vitallium
Copy link
Collaborator Author

@thoop
Copy link

thoop commented Jun 30, 2014

Amazing work! 🍻

@JamesMGreene
Copy link
Collaborator

@vitallium: Are there any built binaries of this commit available?

@vitallium
Copy link
Collaborator Author

@JamesMGreene I have binaries for all platforms. Should I share them?

@JamesMGreene
Copy link
Collaborator

If possible, absolutely, yes. We'll get a lot more testers that way. 👍

@djleeds
Copy link

djleeds commented Jun 30, 2014

Yes, please do share the binaries. Thanks!

@thoop
Copy link

thoop commented Jul 3, 2014

Any updates on sharing those binaries? Would love to start testing!

@ricoli
Copy link

ricoli commented Jul 3, 2014

may I ask which version of ghost driver does it have? :)

@ariya
Copy link
Owner

ariya commented Jul 5, 2014

There seems to be permission/mode issues with some files, when building on OS X.

@ariya
Copy link
Owner

ariya commented Jul 5, 2014

@vitallium I had to skip -icu to build qtbase on OS X. Is it mandatory to install ICU header files to build?

@vitallium
Copy link
Collaborator Author

@ariya yes. ICU is required to build QtWebkit.

@jiweigang1
Copy link

@vitallium where can i download the binaries ? thank you

@GehDoc
Copy link

GehDoc commented Jul 7, 2014

@vitallium i managed to build it on Fedora 20 from your repository branch 2.0, but...

There was missing files, and file rights, in QT part, and broken google_breakpad / QT includes on pure phantomjs part.
I may have done something wrong (wrong repository ?). If not, I can send you the list of my changes (not a diff, unless you provide my a way to cleanup my build directory).

@vitallium
Copy link
Collaborator Author

@GehDoc Awesome! Yes, send the the list please! Thanks!

@GehDoc
Copy link

GehDoc commented Jul 8, 2014

@vitallium
Copy link
Collaborator Author

@GehDoc Thanks! Stupid .gitignore file. I'll add missing files.

@KabanaSoft
Copy link

@vitallium - Could you please share the PhantomJS2 binaries? I would like to give them a test, thanks!

@yogu13
Copy link

yogu13 commented Jul 10, 2014

@vitallium will you be sharing the binaries ? I wouldnt mind testing them.

@vitallium
Copy link
Collaborator Author

@ariya ping

@ariya
Copy link
Owner

ariya commented Jul 25, 2014

@vitallium Linux build fails on breakpad. I see that breakpad is also updated in this PR. Any more info regarding this?

@vitallium
Copy link
Collaborator Author

@ariya new version of breakpad contains many fixes. So, I thought this will be a good idea to update it too. But, for now, I think I'll revert it. Should I revert it?

@ariya
Copy link
Owner

ariya commented Jul 25, 2014

@vitallium Reverting will be useful. Then I can continue my Linux testing.

@vitallium
Copy link
Collaborator Author

Ok. I'll revert.

Sent from my iPhone

On 25 Jul 2014, at 22:20, Ariya Hidayat notifications@github.com wrote:

@vitallium Reverting will be useful. Then I can continue my Linux testing.


Reply to this email directly or view it on GitHub.

@vitallium
Copy link
Collaborator Author

Done! @ariya

@ariya
Copy link
Owner

ariya commented Jul 28, 2014

Landed. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.