Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 54016 - [ebuild update] app-vim/cream-0.30
Summary: [ebuild update] app-vim/cream-0.30
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: Low enhancement (vote)
Assignee: Vim Maintainers
URL: http://cream.sourceforge.net/changelo...
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2004-06-15 12:56 UTC by TGL
Modified: 2004-06-22 14:26 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
cream.ebuild--0.29-r1_to_0.30.patch (cream.ebuild--0.29-r1_to_0.30.patch,1.01 KB, patch)
2004-06-15 12:57 UTC, TGL
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description TGL 2004-06-15 12:56:28 UTC
Hi Ciaranm,

Cream 0.30 has been released. It's a minor bugfixes release. The good thing is that we don't have anything to patch anymore (but the installation path).

I will submit the ebuild update as a patch against the 0.29-r1 one.

Thanks,
Thomas.
Comment 1 TGL 2004-06-15 12:57:27 UTC
Created attachment 33336 [details, diff]
cream.ebuild--0.29-r1_to_0.30.patch
Comment 2 Ciaran McCreesh 2004-06-15 15:39:01 UTC
Thanks very much for the ebuild.

Hm. Just noticed this when exiting:

(gvim:1541): GLib-GObject-CRITICAL **: file gobject.c: line 1579 (g_object_unref): assertion `G_IS_OBJECT (object)' failed
Vim: Caught deadly signal SEGV
Vim: Finished.

Happens on earlier versions of both gvim and cream as well. You'll only see it if you run cream from within a term.

I'm going to commit the 0.30 ebuild anyway once my cvs upd is done, since all versions of cream in portage are affected by this. However, we probably want to talk to upstream to see whether this is something we're breaking or whether it's a universal bug.

BTW, it's generally considered ok to leave in ~arch for existing archs when doing relatively minor version bumps.

Leaving the bug open for now until we figure out the SEGV.
Comment 3 TGL 2004-06-15 16:58:02 UTC
The bug you've seen already happened to me, and is not really specific to Cream but can happen as soon as you create a toolbar button with an icon from a bitmap file (ie. not one of the usual icons, which are included in the binary and are loaded by a different piece of code). I've reported it to vim-dev some time ago:
http://marc.theaimsgroup.com/?l=vim-dev&m=108317705302808

The problem is that I don't know for sure whether the code I've replaced in this patch was correct or not (meaning whether it was a gtk bug that i've workaround or a gvim bug that i've fixed). And now I can't reproduce it anymore, which doesn't help... (this may mean that it was a gtk bug fixed in one of the recent updates, but who knows... And i've not found anything relevant in changelogs.) 
What versions of glib/gtk+ are you using?
Comment 4 TGL 2004-06-22 08:16:41 UTC
Ok, I've redone a few tests and can now confirm that the bug was in gtk+ =<2.4.1, and not in gvim. It doesn't occur anymore with gtk+-2.4.3.  

The relevent gtk change that fixed that, and which has been included in gtk+-2.4.2, is here:
http://bugzilla.gnome.org/attachment.cgi?id=27410&action=view

And the patch i've submitted to vim-dev was actually not a fix but only a workaround for this GtkIconSource bug.

I don't know, maybe that implies a dependencies changes in gvim ebuild? Something like changing ">=x11-libs/gtk+-2.2" to "|| ( =x11-libs/gtk+-2.2* >=x11-libs/gtk+-2.4.2 )" maybe? 
That said, since gtk+-2.4.1 is the last stable version on most archs, it's probably not that a good idea.
Comment 5 Ciaran McCreesh 2004-06-22 14:26:02 UTC
Ok, closing this one off since there's not much we can do about the SEGV... At some point I'll be upping the gtk+-2 DEPENDs for the eclass for the new file dialog thingie anyway.

Thanks again for the ebuild.