Bug 348088 - Support for latest Flash plugin
Support for latest Flash plugin
Status: RESOLVED FIXED
: 350515 350525 350545 350564 350656 350703 351877 352873 353581 (view as bug list)
Classification: openSUSE
Product: openSUSE 10.3
Classification: openSUSE
Component: KDE
Final
Other Other
: P5 - None : Major (vote)
: ---
Assigned To: E-mail List
E-mail List
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-12-12 14:38 UTC by Lubos Lunak
Modified: 2008-02-20 07:28 UTC (History)
15 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
kdelibs3 patch (4.79 KB, patch)
2007-12-12 14:40 UTC, Lubos Lunak
Details | Diff
kdebase3 patch (22.76 KB, patch)
2007-12-12 14:42 UTC, Lubos Lunak
Details | Diff
kdebase3 patch (65.89 KB, patch)
2007-12-14 16:31 UTC, Lubos Lunak
Details | Diff
Example crash (4.96 KB, text/plain)
2007-12-18 11:58 UTC, Stephan Binner
Details
kdelibs3 patch (4.84 KB, patch)
2007-12-19 17:57 UTC, Lubos Lunak
Details | Diff
kdebase3 patch (66.27 KB, patch)
2007-12-19 18:00 UTC, Lubos Lunak
Details | Diff
kdelibs3 patch (4.79 KB, patch)
2007-12-21 14:10 UTC, Lubos Lunak
Details | Diff
kdebase3 patch (68.17 KB, patch)
2007-12-21 14:22 UTC, Lubos Lunak
Details | Diff
gtk-qt-engine patch (751 bytes, patch)
2008-01-10 15:24 UTC, Lubos Lunak
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lubos Lunak 2007-12-12 14:38:41 UTC
AKA "Adobe sucks".
The latest Flash player requires so-called "XEmbed browser protocol", and, somewhat forgetting to tell anyone, also requires Gtk2 eventloop support. Without either it simply does not work.
Comment 1 Lubos Lunak 2007-12-12 14:40:53 UTC
Created attachment 187237 [details]
kdelibs3 patch

This patch fixes QXEmbed in kdelibs. This patch should be final.
Comment 2 Lubos Lunak 2007-12-12 14:42:53 UTC
Created attachment 187238 [details]
kdebase3 patch

This is preliminary kdebase3 patch. It should be generally usable, but there are still few outstanding issues:
- the Glib eventloop integration polls, it should be probably fixed not to
- Flash crashes in XRemoteTimeout(), it seems to be a Flash bug, but given that Adobe apparently doesn't give a damn and it's pretty annoying, I guess a workaround is in order
Comment 3 Lubos Lunak 2007-12-14 16:31:20 UTC
Created attachment 187677 [details]
kdebase3 patch

The (hopefully) final version of the patch. I've left the Glib polling there as not being really necessary to fix and not worth it at this time. All the other problems should be fixed (well, that's a guess of course, since flashplayer is closed-source and the main reason for this is probably to prevent people getting sick from looking at parts of its code - there must be horrible things in it).
Comment 4 Lubos Lunak 2007-12-14 16:32:50 UTC
The patch should be tested on a 64bit system, but otherwise it should be ready.
Comment 6 Stephan Binner 2007-12-18 11:55:12 UTC
I don't seem to be able to make it work on x86_64 and on i586 I get occasional rather random crashes (tested on http://wetter.rtl.de/deutschland/dt.php).
Comment 7 Stephan Binner 2007-12-18 11:58:12 UTC
Created attachment 188041 [details]
Example crash
Comment 8 İsmail Dönmez 2007-12-19 11:14:21 UTC
Can't get it to work on KDE 3.5.8 on 32bit either. youtube.com is mostly blank in the places of videos.
Comment 9 Lubos Lunak 2007-12-19 15:45:25 UTC
Works fine on 32bit here, so there's nothing I can do unless you provide more details.
Comment 10 İsmail Dönmez 2007-12-19 15:49:18 UTC
I am seeing 

Adobe Flash Player: gtk_clipboard_get(GDK_SELECTION_PRIMARY); failed. Trying to call gtk_init(0,0);

on konsole, maybe its related.
Comment 11 Lubos Lunak 2007-12-19 15:53:50 UTC
No, that's normal. Are you sure you have both patches? If yes, please attach full debug output.
Comment 12 İsmail Dönmez 2007-12-19 15:57:13 UTC
Sorry didn't notice the kdelibs patch, re-testing now.
Comment 13 İsmail Dönmez 2007-12-19 17:10:37 UTC
Works fine after applying two patches, nice work!
Comment 14 Lubos Lunak 2007-12-19 17:57:40 UTC
Created attachment 188289 [details]
kdelibs3 patch
Comment 15 Lubos Lunak 2007-12-19 18:00:50 UTC
Created attachment 188290 [details]
kdebase3 patch

With these patches even 64b with nspluginwrapper should work. Every Flash applet may be actually running twice because of a Flash bug, but hey, who cares.

I have no idea about the random crashes from comments #6/#7, otherwise I'm not aware of any (serious) problem with the patches.
Comment 16 İsmail Dönmez 2007-12-19 21:38:43 UTC
Last two patches look rather crashy here, I will revert to old patches and try a bit more.
Comment 17 Stephan Binner 2007-12-20 10:07:51 UTC
Doesn't seem to work on x86_64 with the new patches either, and I have also the impression that it got more unstable on i586.
Comment 18 İsmail Dönmez 2007-12-20 13:19:24 UTC
Last two patches made flash crash on 32bit, the good patches are :

 kdebase3 patch   (65.89 KB, patch)
2007-12-14 09:31 MST, Lubos Lunak

 kdelibs3 patch   (4.79 KB, patch)
2007-12-12 07:40 MST, Lubos Lunak

Thanks,
ismail
Comment 21 Lubos Lunak 2007-12-20 17:12:17 UTC
I suggest we release patches from comments #3 and #14 [*] together with the flash update unless I manage to provide better ones.
[*] That is, the latest kdelibs3 patch (attachment #188289 [details]) and the older kdebase3 patch (attachment #187677 [details]).

Comment 22 İsmail Dönmez 2007-12-20 17:46:20 UTC
Those work fine for me, kudos for this work, and shame on Adobe.
Comment 23 Ludwig Nussel 2007-12-21 08:49:28 UTC
who is going to submit kdelibs3 packages?
Comment 25 Lubos Lunak 2007-12-21 14:10:50 UTC
Created attachment 188531 [details]
kdelibs3 patch

Ok, we're back at the original kdelibs3 patch (surprise, surprise). The other one shouldn't hurt though.
Comment 26 Lubos Lunak 2007-12-21 14:22:11 UTC
Created attachment 188532 [details]
kdebase3 patch

This one should even apply cleanly to our packages :).
Comment 28 Lubos Lunak 2007-12-21 18:12:11 UTC
Looks like attachment #188289 [details] as the kdelibs3 patch is a safer bet for now after all.
Comment 32 Marcus Meissner 2007-12-22 13:27:56 UTC
*** Bug 350515 has been marked as a duplicate of this bug. ***
Comment 33 Marcus Meissner 2007-12-22 21:16:06 UTC
*** Bug 350525 has been marked as a duplicate of this bug. ***
Comment 34 Stephan Binner 2007-12-23 19:03:28 UTC
*** Bug 350545 has been marked as a duplicate of this bug. ***
Comment 35 Benjamin Weber 2007-12-23 22:50:50 UTC
*** Bug 350564 has been marked as a duplicate of this bug. ***
Comment 36 Benjamin Weber 2007-12-25 20:41:30 UTC
*** Bug 350656 has been marked as a duplicate of this bug. ***
Comment 37 Marcus Meissner 2007-12-26 16:24:42 UTC
*** Bug 350703 has been marked as a duplicate of this bug. ***
Comment 39 Stephan Binner 2008-01-05 09:56:27 UTC
*** Bug 351877 has been marked as a duplicate of this bug. ***
Comment 40 Marcus Meissner 2008-01-09 12:47:45 UTC
we urgently need sles10 kdebase3 packages for this.
Comment 41 Sven Burmeister 2008-01-09 12:54:11 UTC
Did anyone contact Adobe and asked them whether they would like to avoid these kind of consequences in the future? Otherwise moonlight might really become an alternative if one wants to avoid Adobe's way of treating their KDE users.
Comment 42 Lubos Lunak 2008-01-09 15:29:59 UTC
Hmm. I tried the 10.1 updated packages too and I can confirm they don't work for me either. However, my own build of 3.5 branch, or the 3.5.1 release (i.e. 10.1 KDE), with latest qxembed and nsplugins, do work for me.
So maybe one of our patches is interfering, or the package for 10.1 does not incorporate the fixes properly, or I did something wrong, or something :-/.
Comment 43 Lubos Lunak 2008-01-10 12:51:14 UTC
*** Bug 352873 has been marked as a duplicate of this bug. ***
Comment 44 Lubos Lunak 2008-01-10 15:24:48 UTC
Created attachment 190098 [details]
gtk-qt-engine patch

Patch for gtk-qt-engine for SL10.1/SLED10.
Comment 45 Dirk Mueller 2008-01-10 16:43:08 UTC
added to the package and submitted. gtk-qt-engine needs to be released for 10.1/sle10
Comment 46 Marcus Meissner 2008-01-10 16:50:49 UTC
is a gtk-qt-engine update necessary for 10.2 too?
Comment 47 Marcus Meissner 2008-01-10 16:54:30 UTC
and/or nld9?


kdebase3 packages for SLES 10 and NLD 9 are still missing.
Comment 48 Achim Mildenberger 2008-01-14 09:15:12 UTC
Are there plans to release the kdebase3/kdelib3 patches for 
openSuSE 10.2 as well?
I haven't found them yet or am I missing something?

Thank you.

Comment 49 Marcus Meissner 2008-01-14 09:19:35 UTC
Yes. 

However it is not curing all of the problems, on 64bit I only see Gray where there should be flash applets etc....
Comment 50 Stephan Binner 2008-01-14 09:41:02 UTC
*** Bug 353581 has been marked as a duplicate of this bug. ***
Comment 51 Lubos Lunak 2008-01-15 17:47:02 UTC
Ok, 64bit fixed as well. For the record: The first commit for kdebase/nsplugins is r746927, for kdelibs/kdeui is r747606, and since the fixes break kcmshell in root mode (KDE#155001), also kdebase/kcontrol/kcontrol r757345 should be included.
Comment 52 Lubos Lunak 2008-01-15 17:54:21 UTC
Comment on attachment 188532 [details]
kdebase3 patch

Gtk-qt-engine patch remains, other are in SVN.
Comment 53 Dirk Mueller 2008-01-16 00:25:19 UTC
10.3,10.2,10.1,SLE10,NLD9 submitted (gtk-qt-engine, kdebase3, kdelibs3)

SLE10 and 10.3 has been tested. NLD9 is untested. 
Comment 56 Marcus Meissner 2008-01-18 13:54:33 UTC
released all but the NLD 9 fixes now.
Comment 59 Marcus Meissner 2008-01-23 12:41:38 UTC
so, all released now.