[oe] [meta-oe][PATCH 2/2] eject: Fix wrong return value

[Huang, Jie (Jackie)]

> -----Original Message-----
> From: openembedded-devel-bounces at lists.openembedded.org [mailto:openembedded-devel-
> bounces at lists.openembedded.org] On Behalf Of Khem Raj
> Sent: Wednesday, April 09, 2014 11:49 PM
> To: openembedded-devel at lists.openembedded.org
> Subject: Re: [oe] [meta-oe][PATCH 2/2] eject: Fix wrong return value
> 
> On Apr 8, 2014 10:36 PM, "Huang, Jie (Jackie)" <Jackie.Huang at windriver.com>
> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: openembedded-devel-bounces at lists.openembedded.org [mailto:
> openembedded-devel-
> > > bounces at lists.openembedded.org] On Behalf Of Khem Raj
> > > Sent: Wednesday, April 09, 2014 4:13 PM
> > > To: openembedded-devel at lists.openembedded.org
> > > Subject: Re: [oe] [meta-oe][PATCH 2/2] eject: Fix wrong return value
> > >
> > > On Apr 8, 2014 8:27 PM, <jackie.huang at windriver.com> wrote:
> > > >
> > > > From: Jackie Huang <jackie.huang at windriver.com>
> > > >
> > > > Remove the unnecessary return(1) from function usage(), so that
> > > > eject -h can return the right value.
> > >
> > > this is an API change i wonder if other programs rely on the value
> being >0
> >
> > I think the answer should be no, I don't see any reason to return 1
> > for
> the "-h" or
> > "--help", almost all programs return 0 for that, right?
> >
> 
> mostly yes and change is ok too just discuss it with package community and see if this is ok.

Yeah, sure, I will send it to the community, thanks!

Thanks,
Jackie


> > Thanks,
> > Jackie
> >
> > > >
> > > > Signed-off-by: Zhangle Yang <zhangle.yang at windriver.com>
> > > > Signed-off-by: Jackie Huang <jackie.huang at windriver.com>
> > > > ---
> > > >  .../eject/eject/eject-2.1.5-error-return.patch     |   10 ++++++++++
> > > >  meta-oe/recipes-support/eject/eject_2.1.5.bb       |    4 +++-
> > > >  2 files changed, 13 insertions(+), 1 deletion(-)  create mode
> > > > 100644
> > > meta-oe/recipes-support/eject/eject/eject-2.1.5-error-return.patch
> > > >
> > > > diff --git
> > > a/meta-oe/recipes-support/eject/eject/eject-2.1.5-error-return.patch
> > > b/meta-oe/recipes-support/eject/eject/eject-2.1.5-error-return.patch
> > > > new file mode 100644
> > > > index 0000000..2cce3c3
> > > > --- /dev/null
> > > > +++
> b/meta-oe/recipes-support/eject/eject/eject-2.1.5-error-return.patch
> > > > @@ -0,0 +1,10 @@
> > > > +Upstream-Status: Pending
> > > > +
> > > > +--- eject/eject.c.orig 2013-09-11 18:08:36.000000000 +0800
> > > > ++++ eject/eject.c      2013-09-11 18:09:05.000000000 +0800
> > > > +@@ -207,7 +207,6 @@
> > > > + "If omitted, name defaults to `%s'.\n"
> > > > + "By default tries -r, -s, -f, and -q in order until success.\n"),
> > > > +                       DEFAULTDEVICE);
> > > > +-  exit(1);
> > > > + }
> > > > diff --git
> a/meta-oe/recipes-support/eject/eject_2.1.5.bbb/meta-oe/recipes-support/eject/
> > > eject_2.1.5.bb
> > > > index 98319a3..c47c653 100644
> > > > --- a/meta-oe/recipes-support/eject/eject_2.1.5.bb
> > > > +++ b/meta-oe/recipes-support/eject/eject_2.1.5.bb
> > > > @@ -5,7 +5,9 @@ LIC_FILES_CHKSUM =
> > > "file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b"
> > > >
> > > >  inherit autotools gettext update-alternatives
> > > >
> > > > -SRC_URI = "http://sources.openembedded.org/${BP}.tar.gz"
> > > > +SRC_URI = "http://sources.openembedded.org/${BP}.tar.gz \
> > > > +           file://eject-2.1.5-error-return.patch \ "
> > > >
> > > >  SRC_URI[md5sum] = "b96a6d4263122f1711db12701d79f738"
> > > >  SRC_URI[sha256sum] =
> > > "ef9f7906484cfde4ba223b2682a37058f9a3c7d3bb1adda7a34a67402e2ffe55"
> > > > --
> > > > 1.7.9.5
> > > >
> > > > --
> > > > _______________________________________________
> > > > Openembedded-devel mailing list
> > > > Openembedded-devel at lists.openembedded.org
> > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > > --
> > > _______________________________________________
> > > Openembedded-devel mailing list
> > > Openembedded-devel at lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel