[APPLIED] Pull request for Raring
Andy Whitcroft
apw at canonical.com
Thu Nov 8 17:26:31 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.
Ok, I cannot reproduce this behaviour, and the internal test suite seems
to pass with similar constructions. Odd.
-apw
More information about the kernel-team
mailing list