[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108340 is|| obsolete|| --- Comment #35 from Kevin Kofler --- Comment on attachment 1108340 --> https://bugzilla.redhat.com/attachment.cgi?id=1108340 qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v5 The no-sse2 patch attached here is incomplete (missing at least fixes for WebGL image conversion and for webrtc's use of OpenMAX) and thus obsolete. I am now working on this in dist-git. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Resolution|NOTABUG |DUPLICATE --- Comment #34 from Kevin Kofler --- *** This bug has been marked as a duplicate of bug 1295549 *** -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Alias|qt5-qtwebengine,| |qtwebengine | -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NOTABUG Flags|fedora-review? | |needinfo?(he...@kde.org)| Last Closed||2016-01-04 14:14:10 --- Comment #33 from Kevin Kofler --- The one week is over now, still no response from the submitter, closing as per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews#Submitter_not_responding . I can submit a new review request in a few minutes, I'm just waiting for my Copr build to complete. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #32 from Kevin Kofler --- (The last comment from the submitter was comment #8 at 2015-08-05 21:35:52 EDT.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added CC||he...@kde.org Flags||needinfo?(he...@kde.org) --- Comment #31 from Kevin Kofler --- https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews#Submitter_not_responding > When the submitter of a review ticket has not responded to comments for one > month, a comment is added to the ticket indicating that the review is stalled > and that a response is needed soon. > If there is no response within one week, the ticket is closed with resolution > NOTABUG, and the fedora-review flag is set to the empty value. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #30 from Kevin Kofler --- Created attachment 1109065 --> https://bugzilla.redhat.com/attachment.cgi?id=1109065&action=edit qtwebengine-spec-ffmpeg-clean.patch Here is a patch to the specfile that implements the required FFmpeg cleanup. In addition to patching the specfile, it also creates 3 scripts: clean_ffmpeg.sh and process_ffmpeg_gyp.py are verbatim from spot's Chromium packaging, clean_qtwebengine.sh is a small wrapper around clean_ffmpeg.sh that I wrote for our purposes. The usage is documented in the specfile: wget http://download.qt.io/official_releases/qt/5.5/5.5.1/submodules/qtwebengine-opensource-src-5.5.1.tar.xz ./clean_qtwebengine.sh 5.5.1 → Source0: qtwebengine-opensource-src-5.5.1-clean.tar.xz ./clean_qtwebengine.sh 5.6.0-beta also works (and it accepts both the 7z and the tar.gz: the script looks for a tar.xz, tar.bz2, tar.gz and 7z in that order, others such as zip can easily be added when/if needed; the output is ALWAYS a tar.xz). To generate the SRPM, just run the above 2 commands and rpmbuild -bs qt5-qtwebengine.spec. I haven't uploaded it because it is huge. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #29 from Kevin Kofler --- I did a quick test: No matter what method is used to find libraries (systemwide ld.so search path, LD_LIBRARY_PATH or rpath), ld.so will ALWAYS look for sse2/*.so in that path before taking *.so. What this means for us is that even if we build a shared libv8.so in a private directory, we can provide an sse2/libv8.so override in the same private directory. So this can be used to make it pick the right V8 build at runtime. (This is not currently implemented in the no-sse2 patch, it just builds V8 once with the x87 target.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Depends On|1293190 | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1293190 [Bug 1293190] v8 on i686 should be built with the x87 arch, possibly with ia32 builds in lib/sse2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #28 from Kevin Kofler --- As far as I can tell, what blocks the review is that the source tarball needs to be cleaned using this shell script: http://copr-dist-git.fedorainfracloud.org/cgit/spot/chromium/chromium.git/tree/clean_ffmpeg.sh?id=474ebe047c82342cf37b722681981f6ab2a793da and this Python script that is called by the above: http://copr-dist-git.fedorainfracloud.org/cgit/spot/chromium/chromium.git/tree/process_ffmpeg_gyp.py?id=474ebe047c82342cf37b722681981f6ab2a793da This just needs to be adapted to be run inside the 3rdparty/chromium directory. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108333 is|| obsolete|| --- Comment #27 from Kevin Kofler --- Created attachment 1108340 --> https://bugzilla.redhat.com/attachment.cgi?id=1108340&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v5 This version restores runtime detection for the last 2 places that were missing it: the SSE1 code in media/base. Instead of my previous quick hack that disabled the SSE routines entirely, I reverted review 308003004 properly. (Rebase hint: Most of the code there is just cut from one file and pasted into the other, so if you get a reject on the removal part, just redo the cut&paste and the _sse.cc file will be up to date as well. This is in fact what I did for sinc_resampler.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #26 from Kevin Kofler --- Created attachment 1108333 --> https://bugzilla.redhat.com/attachment.cgi?id=1108333&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v4 (This time hopefully complete. I had accidentally uploaded the patch while diff was still running.) This version fixes src/3rdparty/chromium/third_party/qcms/BUILD.gn to define SSE2_ENABLE (needed to enable SSE2 runtime detection) for x86. qcms.gyp already got this right, I just added a comment there clarifying that SSE2_ENABLE enables runtime detection here. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108332 is|| obsolete|| Attachment|qtwebengine-opensource-src- |[incomplete upload] #1108332|5.6.0-beta-no-sse2.patch v4 | description|| -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108194 is|| obsolete|| --- Comment #25 from Kevin Kofler --- Created attachment 1108332 --> https://bugzilla.redhat.com/attachment.cgi?id=1108332&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v4 This version fixes src/3rdparty/chromium/third_party/qcms/BUILD.gn to define SSE2_ENABLE (needed to enable SSE2 runtime detection) for x86. qcms.gyp already got this right, I just added a comment there clarifying that SSE2_ENABLE enables runtime detection here. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Alias||qtwebengine -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108193 is|| obsolete|| --- Comment #24 from Kevin Kofler --- Created attachment 1108194 --> https://bugzilla.redhat.com/attachment.cgi?id=1108194&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v3 This version fixes a missing closing brace in src/3rdparty/chromium/media/base/yuv_convert.cc (again found through proofreading). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added Attachment|0 |1 #1108190 is|| obsolete|| --- Comment #23 from Kevin Kofler --- Created attachment 1108193 --> https://bugzilla.redhat.com/attachment.cgi?id=1108193&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch v2 This version fixes the MMX stuff in src/3rdparty/chromium/media/base/simd/convert_yuv_to_rgb.h: The assembly code was actually correct because it was still updated by upstream as it is shared with the SSE version, but the C prototypes were not (they were outdated). (I realized this while proofreading the patch.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #22 from Kevin Kofler --- Created attachment 1108190 --> https://bugzilla.redhat.com/attachment.cgi?id=1108190&action=edit qtwebengine-opensource-src-5.6.0-beta-no-sse2.patch The attached (not yet tested in any way) patch should make QtWebEngine run on non-SSE2 x86 CPUs. The patch is a cumulative (forward-ported) revert of upstream reviews 187423002, 511773002 (parts relevant to QtWebEngine only), 516543004, 1152053004 and 1161853008, along with some custom fixes (for example, the third-party qcms supports runtime detection of SSE2, my patch should make this work instead of just disabling SSE2 support entirely as Chromium was doing before they started requiring SSE2). If you think this patch is too large, I can do a version that does not readd the removed (no longer supported by upstream) MMX- and SSE(1)-optimized routines in "media" (those where an SSE2 version exists), always falling back to the plain C ones for non-SSE2 instead. (The patch as it is now restores the full runtime detection, which is IMHO nicer.) The current QtWebEngine only uses gyp, but I tried to also fix the gn setup because I guess it will be used in the future. I did not find the place where gn sets the -msse2 -mfpmath=sse -mmmx flags though, so that will need to be addressed separately when needed. For those who care about the original Chromium, you will also want to revert the remaining parts of review 511773002 and the review 296663002. I cannot do this in this patch because QtWebEngine does not ship those files at all. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #21 from Kevin Kofler --- No wonder the patch against Chromium v38 from https://github.com/bircoph/chromium-no-sse2-patch does not work: It removes -msse2, but not -mfpmath=sse, which also requires SSE2! It is also otherwise hackish, in particular, it completely removes SSE2 support for x86 instead of making it conditional on ENABLE_SSE2. We can definitely do better. The "no-sse4" patch that patches the bundled libvpx is also bogus, because that code (and also the SSE2 code that is also there) is runtime-detected and as such should not be an issue. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Rex Dieter changed: What|Removed |Added Depends On||1293190 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1293190 [Bug 1293190] v8 on i686 should be built with the x87 arch, possibly with ia32 builds in lib/sse2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #20 from Kevin Kofler --- I have just filed bug #1293190: v8 on i686 should be built with the x87 arch, possibly with ia32 builds in lib/sse2. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #19 from Kevin Kofler --- Actually, we don't have to patch V8: Shortly after that announcement, a separate x87 backend (x86 with x87 FP, no SSE2) was added to V8: https://github.com/v8/v8/tree/master/src/x87 So V8 support is actually already there in the source code, we just need the system v8 package to use it (as far as I can see, it currently doesn't) if we are building against system V8. (The last Chromium no-sse2 patch from https://github.com/bircoph/chromium-no-sse2-patch in fact already did set the V8 backend to x87.) There's only the Chromium part to fix, and that's probably not that hard. At least it shouldn't require messing with a JIT compiler. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #18 from Kevin Kofler --- This is the referenced patchset to revert for V8: https://codereview.chromium.org/275433004 The other one: https://codereview.chromium.org/275253004 is about CMOV, but since we already require CMOV in Fedora, that one need not be reverted. But since your packaging apparently uses system V8, I guess I need to file this against the system V8 package. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #17 from Kevin Kofler --- See also: https://groups.google.com/forum/#!topic/v8-users/cIVgTxcDrNI for the V8 part of the story, pointing to 2 revisions that would need to be reverted for a starter. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #16 from Kevin Kofler --- This is an attempt by somebody to restore non-SSE2 support, but it is neither up to date nor complete: https://github.com/bircoph/chromium-no-sse2-patch -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #15 from Kevin Kofler --- For the SSE2 stuff, I found out Chromium had actually supported non-SSE2 until mid-2014: http://code.google.com/p/chromium/issues/detail?id=348761 This contains a bunch of commits that need to be reverted in Chromium itself to make non-SSE2 work. For V8, there was x87 support in there until 2014, it will need to be resurrected. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #14 from Kevin Kofler --- The minimum to do to get a legally redistributable tarball is to run the clean_ffmpeg.sh script (uses process_ffmpeg_gyp.py) from spot's Chromium SRPMs. (That doesn't unbundle or remove FFmpeg, mind you, it just removes the encumbered files from it.) Simpler would be to rm -rf third_party/ffmpeg entirely, if we can get stuff to build without it (but then we need the GStreamer stuff or codec support will be almost nonexistent). This might also be needed, if QtWebEngine is including those reference_build directories too: find . -depth -name reference_build -type d -exec rm -rf {} \; -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #13 from Kevin Kofler --- If we can get it to build with media_use_ffmpeg=0, then that should work (but it will still compile a lot of the default media backend, just without the codecs). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #12 from Kevin Kofler --- For GStreamer, the bad news is that disabling FFmpeg in favor of GStreamer is not supported yet: https://github.com/Samsung/ChromiumGStreamerBackend/issues/3 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #11 from Kevin Kofler --- Until that happens, I think we should treat the V8 portability issue as a non-blocker. Especially, because technically, one could get around the SSE2 issue by excluding i686 entirely, but it is of course better to ship a package that works on many-but-not-all i686 machines than none at all. We should just have a tracker bug for the SSE2 issue, as would be required for ExcludeArch. Maybe even put it on the x86 ExcludeArch tracker, even though we won't actually exclude the arch (because that'd be silly). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #10 from Kevin Kofler --- Wow, another miracle, this time going on at Chromium upstream: https://chromium.googlesource.com/v8/v8/+log/master/src/interpreter (started 2 weeks ago, probably not usable yet). When done, and when it reaches (or is backported by us into) QtWebEngine, this will cover the secondary architectures, and through a similar setup as for QtDeclarative, non-SSE2 x86. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #9 from Kevin Kofler --- There's another showstopper though: According to http://code.google.com/p/v8/ , V8 requires SSE2 on 32-bit x86, which is not allowed by Fedora policy. (Of course, secondary architectures are also screwed, but there is no policy against that.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #8 from Helio Chissini de Castro --- Thanks Kevin I will check it tomorrow -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #7 from Kevin Kofler --- Looks like that code will only work on Fedora ≥ 23 though (needs GStreamer ≥ 1.5.2, F23 has it already, F22 has only 1.4.5). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #6 from Kevin Kofler --- Unbundled FFmpeg is not of any help to us, because FFmpeg is not allowed in Fedora. We need to try to patch this: http://blogs.s-osg.org/announcing-a-new-gstreamer-backend-for-chromium/ into our QtWebEngine builds. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Daniel Vrátil changed: What|Removed |Added CC||dvra...@redhat.com --- Comment #5 from Daniel Vrátil --- News from Akademy: the upstream maintainer managed to unbundle many things directly in QtWebEngine for Qt 5.6, including ffmpeg, which is great news. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #4 from Kevin Kofler --- > v8 - Fedora version is too old Then the BuildRequires: v8-devel is bogus. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 --- Comment #3 from Helio Chissini de Castro --- Update: https://heliocastro.fedorapeople.org/qt5/qt5-qtwebengine.spec https://heliocastro.fedorapeople.org/qt5/qt5-qtwebengine-5.5.0-2.fc22.src.rpm - Echoing command is NOT blocker, although is doing it right now - Boguis requires are removed, but is not blocker at all - Unbundled most of the libraries, with exceptions: ffmpeg - Usage of internal one sqlite3 - internal sqlite3 add some functions still not launched upstream libvpx - Single function not present on Fedora 1.4.0 v8 - Fedora version is too old -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Kevin Kofler changed: What|Removed |Added CC||ke...@tigcc.ticalc.org --- Comment #2 from Kevin Kofler --- * The non-echoing of build commands is a clear review blocker. * The bundled libraries are also a clear review blocker. (You haven't even unbundled those libraries that only require a configure flag to unbundle.) * You also have bogus BuildRequires. In particular, GStreamer, which is NOT used anywhere as far as I can tell. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1244196] Review Request: qt5-qtwebengine - Qt5 - QtWebEngine components
https://bugzilla.redhat.com/show_bug.cgi?id=1244196 Rex Dieter changed: What|Removed |Added Status|NEW |ASSIGNED CC||rdie...@math.unl.edu Blocks||928937 (qt-reviews) Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu Alias||qt5-qtwebengine Flags||fedora-review? --- Comment #1 from Rex Dieter --- this one will be fun. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=928937 [Bug 928937] Qt-related package review tracker -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review