[oe] [meta-oe][PATCH] vim: add recipe for vim-tiny

Paul Barker paul at paulbarker.me.uk
Wed Oct 15 17:45:28 UTC 2014


On 15 October 2014 11:38, Martin Jansa <martin.jansa at gmail.com> wrote:
> On Wed, Oct 15, 2014 at 05:53:31AM -0400, jackie.huang at windriver.com wrote:
>> From: Jackie Huang <jackie.huang at windriver.com>
>>
>> Changes:
>> - split the vim recipe to two files
>> - add bb for vim-tiny based on PACKAGECONFIG defined in .inc file
>> - use trim_version to get VIMDIR
>
> If you really want to revert following 2 patches, then you need to provide
> more justifications for your patch and also confirm that the recipes don't
> stage conflicting files in sysroot anymore (I don't see any change in recipe
> preventing that).
>

Agreed. There may be a use for having both vim and vim-tiny in a
package feed, but I think we need a better way of handling it than
this.

This patch is difficult to fully review as it mixes conceptually
different changes together. If this or something similar does go in
after further discussion, it needs to be split up.

Thanks

-- 
Paul Barker

Email: paul at paulbarker.me.uk
http://www.paulbarker.me.uk



More information about the Openembedded-devel mailing list