[Bug 891461] Review Request: phantomjs - Headless WebKit with a JavaScript API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891461

Ariya Hidayat <ariya.hidayat@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ariya.hidayat@xxxxxxxxx

--- Comment #5 from Ariya Hidayat <ariya.hidayat@xxxxxxxxx> ---
Folks,

Thanks for taking the time to look into packaging this officially!

First of all, let me say than rather than "why is this junk bundled?"
assumption, why not discussing it in a suitable channel? We have a mailing-list
for the discussion and I'm sure a lot of other people (esp. potential
packagers) will benefit a lot from the feedback. I believe such a constructive
input will be appreciated a lot. 

Most of your concerns was not a deliberate or philosophical choice, it's pure
practical as in "we don't have any contributor for packaging this stuff so
let's just do it this way". There's only 24 hours/day and this is the best we
can do given the limited time and other compromises.

Some more details of third party libraries. We're very open to have it built
using the system libraries whenever they are available (see also
http://code.google.com/p/phantomjs/issues/detail?id=415, though nobody got the
time to solve it). This is likely perfect for e.g. mongoose. As for readline vs
linenoise, IIRC it was because readline didn't work for us very well (again,
just start a question on our mailing-list). QCommandLine needs to be replaced,
there's nobody contributing that yet.

I'm not sure CoffeeScript can be decoupled easily. The reason is the module
support. If you just run one script, surely an external coffee-to-js will do
the job, but once modules are used (via requirejs), the hooks need to be right
inside PhantomJS itself.

As for Qt itself, there is some background covered in
http://ariya.ofilabs.com/2012/03/the-evolution-of-phantomjs-build-workflow.html.
If you switch to system Qt, some functionalities might be crippled (including
remote debugging, file upload, web font, JavaScript stack trace) not including
potential crashes. So far there is no sign of Digia producing Qt 4.9 which may
or may not fix those issues. Also, we want to support pure headless (sans X11)
which gets really tricky to be implemented with stock Qt.

Last but not least, we want to include a fresher QtWebKit, see
http://code.google.com/p/phantomjs/issues/detail?id=31 for some details. Not
sure how this can be solved without our current solution so far.


I'll be happy to address more concerns or feel free to start a thread on our
mailing-list.

Thank you!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=P46lHdDDVg&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]