Today the kde team finally released a patch that fixes the most often reported konqueror bug: http://bugs.kde.org/show_bug.cgi?id=68523 I had to tweak the patch a little to be able to add it to the ebuild. It would be great if it was included in kdelibs ebuild,as the bug is very annoying.
Created attachment 48529 [details, diff] The patch by Sarah (I tweaked it a little so it can be included in the ebuild)
Sorry if we didn't applied patches to kde ebuilds. I think that the main reason (at least on my side) is that kdebase ebuilds are big and we don't want to force users to recompile kdebase (or other) for a single patch that for most people won't change nothing. Also for this reason we made the splitted kde ebuilds. In this way applying patch won't take a lot of users time as we'll recompile only the affected program.
Well, I see your point. But the patch could be applied w/o the version bump, so the users who install kde for the first time won't have the bug. As this bug was the most often reported konqueror bug the issue is quite serious.
Applying without version bumping has no sense for me as we'll get different konqueror behaviors and of course it's against the gentoo policy. The right thing to do in this cases is a revision bump. If you say that this bug is reported a lof of times we can also bump the ebuild.
This bug has 27 duplicates, 37 comments and got 190 votes (according to kde bugzilla, http://bugs.kde.org/show_bug.cgi?id=68523) - I think it is pretty much.
Honestly, I thought about adding the fix to a new revision of kdebase, but the patch has not been accepted by kde developers yet, so it's neither applied to HEAD, nor to KDE_3_3_BRANCH. My opinion is that we must wait for them to take actions, before taking action ourselves.
Created attachment 51672 [details, diff] New patch. This is the last patch, that was applied on cvs (see already mentioned bug). As this is very hated bug in kde and we don't know how long to wait for the next release I think it's time to add this patch, so gentoo users could avoid this very nasty problem. So, I think it's time to close this bug :) Peter.
Forgot to mention. This patch should be applied cleanly, I hope... Just add necessary epatch in ebuild. Something like: epatch ${FILESDIR}/konqueror-javascript.crash.patch
Did you ask them to backport the patch to kde-3.3.x?
No. I did not. I've take that patch and applied it to the current kdelibs. Compiled and tested. Everything works for me. No crashes. If you need I can ask them to backport.
Yes, it seems reasonable to ask for a backport for everyone and not for Gentoo only.
.
