[oe] [PATCH][geodegx] geodegx: Linux 2.6.31.4

[Petr Štetiar]

Otavio Salvador <otavio at ossystems.com.br> [2009-11-06 10:09:29]:

> NACK!
> 
> On Fri, Nov 6, 2009 at 8:48 AM, Petr Štetiar <ynezz at true.cz> wrote:
> >
> > Signed-off-by: Petr Štetiar <ynezz at true.cz>
> >
> > diff --git a/conf/checksums.ini b/conf/checksums.ini
> > index 808cc11..225a6f8 100644
> > --- a/conf/checksums.ini
> > +++ b/conf/checksums.ini
> > @@ -16954,6 +16954,10 @@ sha256=d7b9f19b92fd5c693c16cd62f441d051b699f28ec6a175d1b464e58bacd8c78f
> >  md5=84c077a37684e4cbfa67b18154390d8a
> >  sha256=0acd83f7b85db7ee18c2b0b7505e1ba6fd722c36f49a8870a831c851660e3512
> >
> > +[http://kernel.org/pub/linux/kernel/v2.6/linux-2.6.31.4.tar.bz2]
> > +md5=ae7fde9fd0ae0d8ac12a1e1536d6d3ce
> > +sha256=e63041db1ef666ca43293762c8aaec27bd4b54b6d86aee68ebb090c1b66584f4
> > +
> >  [http://kernel.org//pub/linux/kernel/v2.6/linux-2.6.9.tar.bz2]
> >  md5=e921200f074ca97184e150ef5a4af825
> >  sha256=f5dba6366e87e91234d1b0069cfea655b0a4cb37ea97f899226f16998e6ab9f1
> > diff --git a/conf/machine/geodegx.conf b/conf/machine/geodegx.conf
> > index 0ec6782..6350f2d 100644
> > --- a/conf/machine/geodegx.conf
> > +++ b/conf/machine/geodegx.conf
> > @@ -6,8 +6,10 @@ TARGET_ARCH =  "i486"
> >
> >  require conf/machine/include/tune-geode.inc
> >
> > -PREFERRED_PROVIDER_virtual/kernel = "linux-geodegx"
> > -BOOTSTRAP_EXTRA_RDEPENDS = "kernel pciutils udev kernel-modules"
> > +PREFERRED_PROVIDER_virtual/kernel = "linux"
> > +MACHINE_EXTRA_RDEPENDS = "kernel grub"
> > +MACHINE_FEATURES = "kernel26 pci ext2 x86"
> > +MACHINE_KERNEL_VERSION = "2.6"
> 
> You don't need to rdepends on the kernel and bootloader. In my case
> for example, I use a initrd and don't need to have the kernel in the
> image of rootfs; another issue, I use syslinux.

Ok. I'll make my custom machine file.

> Besides, machine changes are another commit; please use one commit for
> kernel recipe and other to the machine changes.

True, I'm sorry, it's my local modification. Would you revert/fix it in .dev, or
should I provide patch? Thanks.

-- ynezz