On Mon, 2013-08-26 at 18:00 -0700, Saul Wold wrote:
> On 08/27/2013 02:43 AM, Li Zhijian wrote:
> > add patch from grep-2.5.1-55.el5.src.rpm and
> > enable configration option of --without-include-regex
> >
> > LSB core-tests failed at /tset/LI18NUX2K.L1/utils/grep-tp/T.grep-tp
> > 5,T.egrep-tp 5
> > which is intend to verify this utility can perform pattern matching in
> > searches
> > without regard to case if -i option is specified.
> >
> > Signed-off-by: Li Zhijian <lizhij...@cn.fujitsu.com>
> > SRC_URI = "${GNU_MIRROR}/grep/grep-${PV}.tar.bz2 \
> > file://uclibc-fix.patch \
> > @@ -13,8 +13,25 @@ SRC_URI = "${GNU_MIRROR}/grep/grep-${PV}.tar.bz2 \
> > file://gettext.patch \
> > file://fix64-int-to-pointer.patch \
> > file://Makevars \
> > - file://grep-CVE-2012-5667.patch \
>
> Why are you removing this patch?
He didn't actually remove it, he just moved it further down the list.
But...
> > file://fix-for-texinfo-5.1.patch \
> > + file://grep-2.5.1-fgrep.patch \
> > + file://grep-2.5.1-bracket.patch \
> > + file://grep-2.5-i18n.patch \
> > + file://grep-2.5.1-oi.patch \
> > + file://grep-2.5.1-manpage.patch \
> > + file://grep-2.5.1-color.patch \
> > + file://grep-2.5.1-icolor.patch \
> > + file://grep-skip.patch \
> > + file://grep-2.5.1-egf-speedup.patch \
> > + file://grep-2.5.1-dfa-optional.patch \
> > + file://grep-2.5.1-tests.patch \
> > + file://grep-2.5.1-w.patch \
> > + file://grep-P.patch \
> > + file://grep-CVE-2012-5667.patch \
> > + file://grep-empty-pattern.patch \
> > + file://grep-bz460641.patch \
> > + file://grep-bz434934.patch \
> > + file://grep-man-label.patch \
... it's fairly hard to believe that all these patches are really
necessary to fix the problem with "grep -i" that's described in the
checkin message. It looks as though a more accurate description of this
change would be something like "grep-2.5.1a: import all patches from
RHEL".
Also, although adding --without-included-regex probably is a good thing
(and probably ought to be done for grep-2.14 as well) it's not very
obvious that this has anything to do with the LSB fix either.
p.
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
