[OE-core] [PATCH] qemu.inc: Fix typo for KERNEL_FEATURES

Paul Eggleton paul.eggleton at linux.intel.com
Fri Sep 13 09:19:51 UTC 2013


On Friday 13 September 2013 10:47:24 Martin Jansa wrote:
> On Thu, Sep 12, 2013 at 11:42:54PM -0700, Saul Wold wrote:
> > Thans to Bjorn Arnelid for finding this
> > 
> > Cc: bjorn.arnelid at xdin.com
> > 
> > [YOCTO #5173]
> > 
> > Signed-off-by: Saul Wold <sgw at linux.intel.com>
> > ---
> > 
> >  meta/conf/machine/include/qemu.inc | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/conf/machine/include/qemu.inc
> > b/meta/conf/machine/include/qemu.inc index a78622d..8557777 100644
> > --- a/meta/conf/machine/include/qemu.inc
> > +++ b/meta/conf/machine/include/qemu.inc
> > @@ -26,4 +26,4 @@ PREFERRED_PROVIDER_virtual/kernel ?= "linux-yocto"
> > 
> >  EXTRA_IMAGEDEPENDS += "qemu-native qemu-helper-native"
> >  
> >  # Provide the nfs server kernel module for all qemu images
> > 
> > -KERNEL_FEATURES_append_pn-linux-yocto = " features/nfsd/nfsd-enable.scc"
> > +KERNEL_FEATURES_append_pn_linux-yocto = " features/nfsd/nfsd-enable.scc"
> 
> Wait a minute, isn't _pn-foo correct override?

It is. Bjorn and I discussed this on IRC already, this patch shouldn't be 
applied. He's switched back to the linux-yocto 3.2 kernel and I guess that 
feature is not available/working there, thus this change did fix his build 
failure as it effectively disabled the line. Perhaps we should fix the 
underlying bug instead? Bruce, any suggestions?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



More information about the Openembedded-core mailing list