> -----Original Message----- > From: openembedded-devel-boun...@lists.openembedded.org > [mailto:openembedded-devel- > boun...@lists.openembedded.org] On Behalf Of Khem Raj > Sent: Wednesday, April 09, 2014 11:49 PM > To: openembedded-devel@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.hu...@windriver.com> > wrote: > > > > > > > > > -----Original Message----- > > > From: openembedded-devel-boun...@lists.openembedded.org [mailto: > openembedded-devel- > > > boun...@lists.openembedded.org] On Behalf Of Khem Raj > > > Sent: Wednesday, April 09, 2014 4:13 PM > > > To: openembedded-devel@lists.openembedded.org > > > Subject: Re: [oe] [meta-oe][PATCH 2/2] eject: Fix wrong return value > > > > > > On Apr 8, 2014 8:27 PM, <jackie.hu...@windriver.com> wrote: > > > > > > > > From: Jackie Huang <jackie.hu...@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.y...@windriver.com> > > > > Signed-off-by: Jackie Huang <jackie.hu...@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@lists.openembedded.org > > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > -- > > > _______________________________________________ > > > Openembedded-devel mailing list > > > Openembedded-devel@lists.openembedded.org > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > -- > > _______________________________________________ > > Openembedded-devel mailing list > > Openembedded-devel@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel -- _______________________________________________ Openembedded-devel mailing list Openembedded-devel@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-devel