GNU bug report logs - #34370
grep 3.3 spurious testsuite failure

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 34370 in the body.
You can then email your comments to 34370 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Andreas Schwab <schwab <at> suse.de>
To: bug-grep <at> gnu.org
Subject: grep 3.3 spurious testsuite failure
Date: Thu, 07 Feb 2019 17:20:36 +0100

https://build.opensuse.org/package/live_build_log/openSUSE:Factory:RISCV/grep/standard/riscv64

FAIL: pcre-jitstack
===================

++ initial_cwd_=/home/abuild/rpmbuild/BUILD/grep-3.3/tests
+++ testdir_prefix_
+++ printf gt
++ pfx_=gt
+++ mktempd_ /home/abuild/rpmbuild/BUILD/grep-3.3/tests gt-pcre-jitstack.XXXX
+++ case $# in
+++ destdir_=/home/abuild/rpmbuild/BUILD/grep-3.3/tests
+++ template_=gt-pcre-jitstack.XXXX
+++ MAX_TRIES_=4
+++ case $destdir_ in
+++ destdir_slash_=/home/abuild/rpmbuild/BUILD/grep-3.3/tests/
+++ case $template_ in
++++ unset TMPDIR
+++ d=/home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
+++ case $d in
+++ :
+++ test -d /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
++++ ls -dgo /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
+++ perms='drwx------ 2 4096 Feb  7 15:53 /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX'
+++ case $perms in
+++ :
+++ echo /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
+++ return
++ test_dir_=/home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
++ cd /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
++ gl_init_sh_nl_='
'
++ IFS=' 
'
++ for sig_ in 1 2 3 13 15
+++ expr 1 + 128
++ eval 'trap '\''Exit 129'\'' 1'
+++ trap 'Exit 129' 1
++ for sig_ in 1 2 3 13 15
+++ expr 2 + 128
++ eval 'trap '\''Exit 130'\'' 2'
+++ trap 'Exit 130' 2
++ for sig_ in 1 2 3 13 15
+++ expr 3 + 128
++ eval 'trap '\''Exit 131'\'' 3'
+++ trap 'Exit 131' 3
++ for sig_ in 1 2 3 13 15
+++ expr 13 + 128
++ eval 'trap '\''Exit 141'\'' 13'
+++ trap 'Exit 141' 13
++ for sig_ in 1 2 3 13 15
+++ expr 15 + 128
++ eval 'trap '\''Exit 143'\'' 15'
+++ trap 'Exit 143' 15
++ trap remove_tmp_ 0
+ path_prepend_ ../src
+ test 1 '!=' 0
+ path_dir_=../src
+ case $path_dir_ in
+ abs_path_dir_=/home/abuild/rpmbuild/BUILD/grep-3.3/tests/../src
+ case $abs_path_dir_ in
+ PATH=/home/abuild/rpmbuild/BUILD/grep-3.3/tests/../src:/home/abuild/rpmbuild/BUILD/grep-3.3/src:./src:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin
+ create_exe_shims_ /home/abuild/rpmbuild/BUILD/grep-3.3/tests/../src
+ case $EXEEXT in
+ return 0
+ shift
+ test 0 '!=' 0
+ export PATH
+ require_pcre_
+ echo .
+ grep -P .
.
+ compare /dev/null err
+ compare_dev_null_ /dev/null err
+ test 2 = 2
+ test x/dev/null = x/dev/null
+ test -s err
+ return 0
+ return 0
+ for p in 'base64 -d' 'base64 -D' 'openssl base64 -d' "perl -MMIME::Base64 -0777ne 'print decode_base64(\$_)'" FAIL
+ test 'base64 -d' = FAIL
++ echo eA==
+ x=x
+ test Xx = Xx
+ eval 'b64_decode() { base64 -d; }'
+ break
+ foo=foo
+ test Xfoo = Xfoo
+ fail=0
+ b64_decode
+ base64 -d
+ gzip -d pcrejit.txt
+ LC_ALL=C
+ grep -P -n '^([/](?!/)|[^/])*~/.*' pcrejit.txt
grep: stack overflow
+ test 2 '!=' 1
+ ulimit -s unlimited
+ ulimit -s unlimited
+ returns_ 1 env LC_ALL=C grep -P -n '^([/](?!/)|[^/])*~/.*' pcrejit.txt
grep: stack overflow
+ fail=1
+ Exit 1
+ set +e
+ exit 1
+ exit 1
+ remove_tmp_
+ __st=1
+ cleanup_
+ :
+ test '' = yes
+ cd /home/abuild/rpmbuild/BUILD/grep-3.3/tests
+ chmod -R u+rwx /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
+ rm -rf /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
+ exit 1
FAIL pcre-jitstack (exit status: 1)

-- 
Andreas Schwab, SUSE Labs, schwab <at> suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."

Message #8 received at 34370 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Andreas Schwab <schwab <at> suse.de>
Cc: 34370 <at> debbugs.gnu.org
Subject: Re: bug#34370: grep 3.3 spurious testsuite failure
Date: Sun, 17 Feb 2019 17:51:13 -0800

[Message part 1 (text/plain, inline)]

On Thu, Feb 7, 2019 at 8:23 AM Andreas Schwab <schwab <at> suse.de> wrote:
> https://build.opensuse.org/package/live_build_log/openSUSE:Factory:RISCV/grep/standard/riscv64
...
> + grep -P -n '^([/](?!/)|[^/])*~/.*' pcrejit.txt
> grep: stack overflow
> + test 2 '!=' 1
> + ulimit -s unlimited
> + ulimit -s unlimited
> + returns_ 1 env LC_ALL=C grep -P -n '^([/](?!/)|[^/])*~/.*' pcrejit.txt
> grep: stack overflow
> + fail=1
> + Exit 1
> + set +e
> + exit 1
> + exit 1
> + remove_tmp_
> + __st=1
> + cleanup_
> + :
> + test '' = yes
> + cd /home/abuild/rpmbuild/BUILD/grep-3.3/tests
> + chmod -R u+rwx /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
> + rm -rf /home/abuild/rpmbuild/BUILD/grep-3.3/tests/gt-pcre-jitstack.j5OX
> + exit 1
> FAIL pcre-jitstack (exit status: 1)

Hi Andreas,
Thanks for reporting that.
I am unable to reproduce that on a few systems, but suspect the
attached patch will fix it. Can you confirm?

[test-pcre-jitstack.diff (application/octet-stream, attachment)]

Message #11 received at 34370 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Schwab <schwab <at> suse.de>
To: Jim Meyering <jim <at> meyering.net>
Cc: 34370 <at> debbugs.gnu.org
Subject: Re: bug#34370: grep 3.3 spurious testsuite failure
Date: Mon, 18 Feb 2019 10:23:15 +0100

On Feb 17 2019, Jim Meyering <jim <at> meyering.net> wrote:

> I am unable to reproduce that on a few systems, but suspect the
> attached patch will fix it. Can you confirm?

https://build.opensuse.org/package/show/home:Andreas_Schwab:Factory/grep

Works fine.

Thanks, Andreas.

-- 
Andreas Schwab, SUSE Labs, schwab <at> suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."

Message #14 received at 34370 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Andreas Schwab <schwab <at> suse.de>
Cc: 34370 <at> debbugs.gnu.org
Subject: Re: bug#34370: grep 3.3 spurious testsuite failure
Date: Mon, 18 Feb 2019 10:54:57 -0800

On Mon, Feb 18, 2019 at 1:23 AM Andreas Schwab <schwab <at> suse.de> wrote:
>
> On Feb 17 2019, Jim Meyering <jim <at> meyering.net> wrote:
>
> > I am unable to reproduce that on a few systems, but suspect the
> > attached patch will fix it. Can you confirm?
>
> https://build.opensuse.org/package/show/home:Andreas_Schwab:Factory/grep
>
> Works fine.

Thanks.
Pushed with two changes:
- corrected spelling of your name in the commit log;
- make the test print saved stderr if it was not a stack overflow

I also pushed this:

    build: avoid build failure with --enable-gcc-warnings
    * src/kwset.c (bmexec_trans): Define with _GL_ATTRIBUTE_PURE,
    per suggestion from recent gcc snapshot.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.