Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 48960 - [update] app-vim/cream-0.29
Summary: [update] app-vim/cream-0.29
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Ciaran McCreesh
URL: http://cream.sourceforge.net/changelo...
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2004-04-25 12:09 UTC by TGL
Modified: 2004-04-29 08:17 UTC (History)
1 user (show)

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


Attachments
cream-0.29.ebuild (cream-0.29.ebuild,1.96 KB, text/plain)
2004-04-25 12:09 UTC, TGL
Details
files/cream-0.29--gentoo.patch (cream-0.29--gentoo.patch,2.99 KB, patch)
2004-04-25 12:10 UTC, TGL
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description TGL 2004-04-25 12:09:00 UTC
Hi,

Cream 0.29 has been released.  I've made a few changes to the ebuild: 
 - added the optionnal spell dicts from this page: 
   http://cream.sourceforge.net/spellcheck.html
   This depends on the "spell" USE flag and LINGUAS variable (see 
   explanations in the ebuild).
 - changed "ctags" for "exuberant-ctags" where needed, and added an 
   rdep on dev-utils/ctags
 - added .desktop file and icon
 - fixed a new bug that prevents creation of user's dirs (~/.cream and 
   its subdirs) because of a slash/backslash confusion

Note that for some reason, i get a lot of assert warnings from glib when i close Cream (namely some "file gobject.c: line 1579 (g_object_unref): assertion `G_IS_OBJECT (object)' failed"). It seems to work fine despite of that (this is on as up-to-date ~x86 system, using gvim-6.2-r8 and glib-2.4.0).
Comment 1 TGL 2004-04-25 12:09:55 UTC
Created attachment 30031 [details]
cream-0.29.ebuild
Comment 2 TGL 2004-04-25 12:10:35 UTC
Created attachment 30032 [details, diff]
files/cream-0.29--gentoo.patch
Comment 3 Ciaran McCreesh 2004-04-26 06:38:02 UTC
Thanks. I'm afraid I might not be able to look at this properly until tomorrow -- I'm not ignoring you, I just don't multitask very well :) Sorry...
Comment 4 TGL 2004-04-28 11:54:34 UTC
Don't worry, there's no hurry. It usually takes me more than one month before i start feeling ignored. :)

About the glib warnings i've mentionned above, it is definitely not a Cream bug, but more a GVim (or maybe Gtk-2.4) one. I've reported an attempt of fix/workaround on vim-dev:
http://marc.theaimsgroup.com/?l=vim-dev&m=108317705302808
Comment 5 TGL 2004-04-28 21:31:03 UTC
I just realized that in fact cream has been bumped in portage 2 days after i opened this bug. :)

wmertens, i CC you since you made this bump, in case you want to integrate some of the changes/fix from my version. At least the ctags fix is needed. Also, more patching of INSTALL.sh is needed if you want to use it instead of an src_install rewrite: currently, it install cream in vim62, but vim-6.3 beta will come soon, and would break existing cream installations. 
Comment 6 Ciaran McCreesh 2004-04-28 23:07:00 UTC
0.29 as-is pulled from the tree, as it is broken. 0.29-r1 added in as ~arch.

I made a few changes to the contributed ebuild:

* Don't use the spell? flag. We're not altering the installed app, so we shouldn't be using flags here.
* Split the patches up into three to make them a bit more maintainable.
* Change the src_unpack logic a bit.
* Add in a flashy info about LINGUAS.

I'm not using INSTALL.sh because it's not even remotely destdir-aware. Although it doesn't matter for now, it could very easily create subtle but nasty errors in the future.

Cheers.
Comment 7 TGL 2004-04-29 06:32:54 UTC
Thanks Ciaran for this changes, they're all good. 

There's just a small error in pkg_postinst:
  /usr/sbin/ebuild.sh: line 76: [: too many arguments

Patch:
- if [ -z ${LINGUAS} ] ; then
+ if [ -z "${LINGUAS}" ] ; then
Comment 8 Ciaran McCreesh 2004-04-29 08:17:43 UTC
D'oh! Strange no-one noticed that, enough people looked over the ebuild :) Fixed, cheers.