[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 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?

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