at 2013-8-27 18:43, Phil Blundell wrote:
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".
Agree, you are right.
It is easier for me to import all patches from RHEL rather than analysising
which
are problem fixed patches.
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.
This option --without-included-regex at grep-2.5.1a is also necessary to fix
LSB core-test
Otherwise /tset/LI18NUX2K.L1/utils/grep-tp/T.grep-tp 5,T.egrep-tp 5 would FAIL。
Thanks
Li
p.
--
Best regards.
Li Zhijian (8555)
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
