[APPLIED] Pull request for Raring

Andy Whitcroft apw at canonical.com
Thu Nov 8 17:10:14 UTC 2012


On Thu, Nov 08, 2012 at 07:41:24AM -0500, Ben Collins wrote:
> On Nov 8, 2012, at 5:17 AM, Andy Whitcroft <apw at canonical.com> wrote:
> 
> > On Wed, Nov 07, 2012 at 10:29:13PM -0500, Ben Collins wrote:
> >> Please pull for this commit to master-next:
> >> 
> >> commit f808fd0abdb75313167cdc145a2fb947a4e625b8
> >> Author: Ben Collins <ben.c at servergy.com>
> >> Date:   Wed Nov 7 11:49:28 2012 -0500
> >> 
> >>    UBUNTU: [Config] Update enforce rule for CONFIG_NVRAM to better suit flavours
> >> 
> >>    Signed-off-by: Ben Collins <ben.c at servergy.com>
> >> 
> >> From git://github.com/benmcollins/ubuntu-raring-powerpc.git master-next
> > 
> > I have pulled this in.  Note I have modified the NVRAM check as below to
> > match what I believe you intended (based on the assumption the configs
> > as generated now are correct) as in its previous form the check was
> > effectivly unchecked on non-power:
> > 
> >    !exists CONFIG_NVRAM | \
> >     (arch powerpc &/ value CONFIG_NVRAM y) | \
> >     value CONFIG_NVRAM m
> > 
> > Yes I know the form is somewhat opaque :/.
> > 
> > -apw
> 
> 
> Thanks
> 
> Mainly the problem is that the flavour token does not actually allow for more than one flavour to be listed. So the previous check had "(flavour foo bar \& ...)" and it would fail (sometimes) because it seemed to be evaluating bar as it's own predicate. Not sure how this happens, but I didn't feel like digging around in config-check too much.

Hrm, i'll have a look at that, that would be a bug indeed.

-apw




More information about the kernel-team mailing list