Re: [PATCH 01/19] gvfs: remove invalid PACKAGECONFIG[cdda]

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

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

On Thu, Sep 17, 2015 at 09:07:00AM +0200, Andreas Müller wrote:
> On Thu, Sep 17, 2015 at 8:54 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > On Thu, Sep 17, 2015 at 02:41:13AM +0200, Andreas Müller wrote:
> >> Enabling cdda requires libcdio-paranoia not libcdio. libcdio-paranoia is not
> >> found in layer list so disable cdda.
> >
> > It wouldn't be the first case where PACKAGECONFIG requires dependency
> > which cannot be satisfied, would you mind adding libcdio-paranoia to
> > PACKAGECONFIG for cdda and keeping it by default disabled?
> >
> > Maybe with comment saying that for cdda you need to create/find
> > libcdio-paranoia recipe.
> >
> OK I can do. I think the easiest way to handle is ignoring this commit
> and do the change next series. Please don't block the others (if there
> aren't other complaints) because I am loosing overview what was sent
> and what needs further rebasement.

OK, I can even update this patch for you, because I like your patches
and the rest looks good to me.

Thanks for fixes, I fully agree that in current state it's hard to track
what's broken and by what, it's big mess, hopefully we'll get failures
down when oe-core is stabilized for release.

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

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