Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 61158 - app-vim/gentoo-syntax VS. app-vim/cream
Summary: app-vim/gentoo-syntax VS. app-vim/cream
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High minor
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-21 08:41 UTC by TGL
Modified: 2004-08-26 12:09 UTC (History)
0 users

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


Attachments
cream-0.30--filetype_fix.patch (cream-0.30--filetype_fix.patch,1.45 KB, patch)
2004-08-21 08:42 UTC, TGL
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description TGL 2004-08-21 08:41:47 UTC
Maybe it's a bit early to bug report this since app-vim/gentoo-syntax is not in portage, but i've tried the 20040820 version from Ciaran's overlay, and it triggered a bug in Cream. Basically, the "set filetype=..." in "ftdetect/gentoo.vim" produces several "E218: autocommand nesting too deep" errors when opening an ebuild file in Cream. 

I will attach a fix for cream-0.30. In case app-vim/gentoo-syntax reachs portage, then cream ebuild should probably be revision-bumped with that patch. I've also reported upstream, so in case cream-0.31 is released before gentoo-syntax, there will be probably no real need for such a bugfix revision on 0.30.
Comment 1 TGL 2004-08-21 08:42:28 UTC
Created attachment 37868 [details, diff]
cream-0.30--filetype_fix.patch
Comment 2 Ciaran McCreesh 2004-08-21 08:44:05 UTC
Which cream are you running? 0.30 or 0.30-r1?
Comment 3 TGL 2004-08-21 08:51:22 UTC
0.30-r1
Comment 4 Ciaran McCreesh 2004-08-21 09:08:12 UTC
Aah, ok, now I get it. Running filetype on a filetype autocmd seems rather strange to me. Wouldn't they be better off using the after/ syntax stuff?
Comment 5 Ciaran McCreesh 2004-08-26 12:09:07 UTC
I just added gentoo-syntax to the tree, so looks like we need this. Fix in cream-0.30-r2, thanks very much.