Re: [meta-oe][PATCH v2] lvm2: Add dependency on bash

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 999 bytes --]

On Tue, Jul 30, 2013 at 12:15:19PM +0100, Paul Eggleton wrote:
> On Friday 26 July 2013 19:43:45 Martin Jansa wrote:
> > On Fri, Jul 26, 2013 at 05:25:29PM +0300, Mihai Prica wrote:
> > > Fixes rootfs generation error if bash is not included in the image.
> > 
> > With rpm right?
> 
> If it's what you're suggesting technically it's broken with any package 
> backend, to some degree.

Agreed, I was just commenting on it because "rootfs generation error if
bash is not included" is quite vague and it could be also bashism in
postinst script.

> > Find which script it has in shebang and add bash only to that package
> > RDEPENDS, if it's something which isn't always needed put it in separate
> > package.
> 
> Or if practical patch the script(s) to remove the bashisms and change the 
> shebang to #!/bin/sh instead.
> 
> Cheers,
> Paul
> 
> -- 
> 
> Paul Eggleton
> Intel Open Source Technology Centre

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]