[oe] [PATCH] gawk-native_3.1.4.bb: Removed legacy style staging

[Fahad]

On Wed, 2010-08-18 at 10:40 +0200, Frans Meulenbroeks wrote:
> 2010/8/18 Fahad Usman <fahad_usman at mentor.com>:
> > * converted do_stage to do_install
> >
> > * replaced ${STAGING_BINDIR} with ${D}${bindir}
> >
> > * added NATIVE_INSTALL_WORKS = "1"
> >
> > * added PR = "r1"
> >
> > Signed-off-by: Fahad Usman <fahad_usman at mentor.com>
> > ---
> >  recipes/gawk/gawk-native_3.1.4.bb |    9 ++++++---
> >  1 files changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/recipes/gawk/gawk-native_3.1.4.bb b/recipes/gawk/gawk-native_3.1.4.bb
> > index 0a33f71..67d254c 100644
> > --- a/recipes/gawk/gawk-native_3.1.4.bb
> > +++ b/recipes/gawk/gawk-native_3.1.4.bb
> > @@ -1,5 +1,8 @@
> >  INHIBIT_AUTOTOOLS_DEPS = "1"
> >  require gawk_${PV}.bb
> > +PR = "r1"
> > +
> > +NATIVE_INSTALL_WORKS = "1"
> >
> >  inherit native
> >
> > @@ -8,7 +11,7 @@ PATCHTOOL = "patch"
> >
> >  S = "${WORKDIR}/gawk-${PV}"
> >
> > -do_stage() {
> > -       install -d ${STAGING_BINDIR}
> > -       install -m 755 gawk ${STAGING_BINDIR}
> > +do_install() {
> > +       install -d ${D}${bindir}
> > +       install -m 755 gawk ${D}${bindir}
> >  }
> > --
> 
> I suggest to merge this with the non-native recipe.
> Also I feel that the default do_install should work here, so a
> specific do_install seems not needed (it is an autotools recipe).
> 
> Frans
> 

Hi Frans,
With this do_install function, I get only one binary file
(sysroots/i686-linux/usr/bin/gawk) in the resulting .ipk file, but if I
remove the do_install function from the recipe and let the autotools's
do_install execute, a bunch of other binary files end up in the .ipk
file along with a lot of other stuff. Is it okay to have extra files in
the ipk package or should I leave the do_install function in the recipe.

Secondly, is there a way to merge native and non-native recipes if the
native one need a different do_install function?

Fahad