[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

--- Comment #6 from Dan Callaghan <dcallagh@xxxxxxxxxx> ---
(In reply to comment #5)

Thanks for joining the conversation here, Ariya.

My comment #2 about "bundling junk" was rude and uncivil, I apologise for that.
I was frustrated after spending the morning trying to package linenoise.
Actually my frustration was mainly with linenoise, for actively encouraging
embedding in other programs and not building into a shared library.

I do understand that there are many good reasons for you to bundle these third
party dependencies, but Fedora has very strong policies against bundled
dependencies (also for very good reasons). It's possible to apply for
exemptions from FESCO but I doubt any of the reasons here would be sufficient,
except perhaps for coffee-script due to the module loading issues (and
ghostdriver, which I don't consider to be a third-party dependency at all).

I'm also aware there will be some missing functionality with stock Qt. Right
now I have 20 failures from the test suite, I think the most serious problem is
file uploads, I am still hoping to find a solution for that. The test suite was
also triggering a crash in qtwebkit but I filed that as bug 891464 and it is
already fixed.

I am very keen to see phantomjs shipped in Fedora because I think it's an
excellent tool, but the only way that will happen is if I can find a way to get
it building against packaged versions of all its dependencies.

-- 
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=y2z7jEiEh1&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]