GNU bug report logs - #15690
new snapshot available: grep-2.14.51-7a35

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 15690 in the body.
You can then email your comments to 15690 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: Jim Meyering <jim <at> meyering.net>
To: bug-grep <at> gnu.org
Cc: Benno Schulenberg <coordinator <at> translationproject.org>,
 platform-testers <at> gnu.org
Subject: new snapshot available: grep-2.14.51-7a35
Date: Tue, 22 Oct 2013 22:12:18 -0700

Paul and I have made build/portability-related fixes, and I've
regenerated build files using newer versions of autoconf and automake.
 Also, several translations have been updated.  Thanks to all.  I
think we're probably ready to release, but will wait at least a day or
so more.

grep snapshot:
  http://meyering.net/grep/grep-ss.tar.xz      1.3 MB
  http://meyering.net/grep/grep-ss.tar.xz.sig
  http://meyering.net/grep/grep-2.14.51-7a35.tar.xz

Changes in grep since 2.14.46-f4d35:

Jim Meyering (2):
      maint: remove now-unused wcscoll module
      gnulib: update to latest

Paul Eggert (3):
      build: port to platforms that predefine _FORTIFY_SOURCE
      build: port shell pattern to Solaris 10
      build: avoid chatter from Automake 1.14


Changes in gnulib since 2.14.46-f4d35:

* gnulib e485bba...974b76b (70):
  > regex: also remove dependency on HAVE_WCSCOLL
  > xfreopen: fix typo: s/frepoen/freopen/
  > regex: don't depend on wcscoll
  > error: add the printf attribute to a static function
  > error: make the module depend on vfprintf-posix
  > fpending, obstack, strerror-override: use pure+const function attrs
  > extern-inline: make safe for -Wundef usage
  > mkfifo-tests, etc.: allow HP-UX 11.11 bug
  > acl: allow cross-compilation to Gentoo
  > mgetgroups: remove dependency on realloc-gnu
  > regex-tests: port to HP-UX 11.11
  > verify: document some 'assume' pitfalls
  > strtoumax: fix another typo in previous commit
  > strtoumax: fix typo in previous commit.
  > strtoumax: port to Solaris 8
  > autoupdate
  > strtoimax, strtoumax: port to HP-UX 11.11
  > strtoimax: port to HP-UX 11.11
  > New module 'count-trailing-zeros'.
  > count-leading-zeros: port to MSC; support types wider than 64 bits
  > count-one-bits: port to MSC; support types wider than 64 bits
  > mountlist: fix resource leak with MOUNTED_INTERIX_STATVFS
  > tests: improve diagnostic when an assertion fails
  > verify: new macro 'assume'
  > autoupdate
  > dup2, dup3: work around another cygwin crasher
  > getdtablesize: work around cygwin issue
  > pmccabe2html: escaping of special characters
  > manywarnings: enable nicer gcc warning messages
  > timespec: use the new TIMESPEC_RESOLUTION in a few more places
  > warnings: port --enable-gcc-warnings to Solaris Studio 12.3
  > configmake: support new --runstatedir option
  > ctype, string: depend on extern-inline
  > userspec: support optional parameters to parse_user_spec()
  > timespec: new function make_timespec, and new constants
  > stdio: OS X port of putc_unlocked + extern inline
  > signal: OS X port of sigaddset etc. + extern inline
  > extern-inline: do not always suppress extern inline on OS X
  > extern-inline: document fixes for ctype and wctype macros
  > fflush, freadahead, fseeko: Fix for Android
  > pmccabe2html: fix portability issues
  > getgroups: statement without effect
  > autoupdate
  > update from texinfo
  > update from texinfo
  > glob: fix compilation
  > glob: fix build for platforms without __THROW
  > autoupdate
  > regex-quote: fix buffer access out of bounds
  > glob: avoid -Wattribute warnings on glibc
  > headers: check that _GL_INLINE_HEADER_BEGIN is defined
  > bootstrap: remove the --version requirement from ancillary tools
  > gc: support HMAC-SHA256 and HMAC-SHA512.
  > gettext: update to version 0.18.3.1
  > selinux-at: omit unnecessary include
  > autoupdate
  > d-ino: avoid false negative on symlink
  > update from texinfo
  > autoupdate
  > bootstrap: port to OpenBSD sed
  > warnings: minor optimization
  > warnings: check -Wfoo rather than -Wno-foo
  > accept doclicense option
  > accept doclicense option
  > xvasprintf-tests: port to GCC with hardening flags
  > fpending: port to recent Cygwin change to stdio_ext.h
  > no need for tendentious comment
  > regenerate
  > autoupdate
  > sys_time: port to OpenBSD

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: bug-grep <at> gnu.org
Subject: Re: bug#15690: new snapshot available: grep-2.14.51-7a35
Date: Tue, 22 Oct 2013 23:33:36 -0700

That tarball is built with texinfo 4.13;
perhaps upgrade to texinfo 5.2?  Among other
things that will cause the .info files to quote
'like this' rather than `like this'.

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Jim Meyering <jim <at> meyering.net>, 15690 <at> debbugs.gnu.org
Cc: platform-testers <at> gnu.org
Subject: meyering grep-2.14.51-7a35 fails on Ubuntu 13.10 x86-64
Date: Wed, 23 Oct 2013 00:44:38 -0700

>   http://meyering.net/grep/grep-2.14.51-7a35.tar.xz

That snapshot fails "make check" on Ubuntu 13.10 x86-64.
However, when I generate a tarball from the same git source,
"make check" works for me.

I think I tracked this down to a discrepancy in the tools used
to build this snapshot.  But it's getting late so I hope someone
else can check this.

The snapshot fails because the three tests multibyte-white-space,
pcre-utf8, surrogate-pair all fail.  And they fail because
"printf '\x40'" prints "\x40" rather than "@".
And this happens because the tests are run with
/bin/sh, not /bin/bash; on Ubuntu /bin/sh is Dash
and Dash mishandles printf with '\x...'.  And this
happens because './configure' decides to make SHELL
/bin/sh rather than /bin/bash.  And I think *this*
happens because 'configure' was built with Autoconf
2.69.112-f181 rather than Autoconf 2.69 (this is
the Autoconf 2.69 shipped with Fedora 19).  The
respective 'configure' scripts are pretty hairy,
though, and I can't easily see why the meyering.net
snapshot went awry.

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: "bug-grep <at> gnu.org" <bug-grep <at> gnu.org>
Subject: Re: [platform-testers] new snapshot available: grep-2.14.51-7a35
Date: Wed, 23 Oct 2013 09:51:52 +0200

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

Hi Jim,

Am 23.10.2013 um 07:12 schrieb Jim Meyering <jim <at> meyering.net>:
> Paul and I have made build/portability-related fixes, and I've
> regenerated build files using newer versions of autoconf and automake.
> Also, several translations have been updated.  Thanks to all.  I
> think we're probably ready to release, but will wait at least a day or
> so more.
> 
> grep snapshot:
>  http://meyering.net/grep/grep-ss.tar.xz      1.3 MB
>  http://meyering.net/grep/grep-ss.tar.xz.sig
>  http://meyering.net/grep/grep-2.14.51-7a35.tar.xz
> 
> Changes in grep since 2.14.46-f4d35:
> 
> Jim Meyering (2):
>      maint: remove now-unused wcscoll module
>      gnulib: update to latest
> 
> Paul Eggert (3):
>      build: port to platforms that predefine _FORTIFY_SOURCE
>      build: port shell pattern to Solaris 10
>      build: avoid chatter from Automake 1.14


I just tested on Solaris 10 Sparc with Sun Studio 12. First I get
two warnings:

  cc: Warning: illegal option -fdiagnostics-show-option
  cc: Warning: illegal option -funit-at-a-time

Maybe these should only be added when gcc is detected?

Two tests are failing, first is ere:

> FAIL: ere
> =========
> 
> ++ initial_cwd_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> ++ fail=0
> +++ testdir_prefix_
> +++ printf gt
> ++ pfx_=gt
> +++ mktempd_ /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests gt-ere.XXXX
> +++ case $# in
> +++ destdir_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> +++ template_=gt-ere.XXXX
> +++ MAX_TRIES_=4
> +++ case $destdir_ in
> +++ case $template_ in
> ++++ unset TMPDIR
> +++ d=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> +++ case $d in
> +++ test -d /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++++ ls -dgo /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++++ tr S -
> +++ perms='drwx------   2       2 Oct 23 09:46 /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b'
> +++ case $perms in
> +++ test 0 = 0
> +++ echo /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> +++ return
> ++ test_dir_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++ cd /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++ 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/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $abs_path_dir_ in
> + PATH=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/src:/home/dam/mgar/pkg/.buildsys/v2/gar/bin/sos12-wrappers:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/opt/csw/bin:/opt/csw/bin:/opt/csw/sbin:/opt/csw/sbin:/opt/SUNWspro/bin:/home/dam/mgar/pkg/.buildsys/v2/gar/bin:/usr/bin:/usr/sbin:/usr/java/bin:/usr/ccs/bin:/usr/openwin/bin
> + create_exe_shims_ /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $EXEEXT in
> + return 0
> + shift
> + test 0 '!=' 0
> + export PATH
> + fail=0
> + gawk -f /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/ere.awk /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/ere.tests
> + /bin/sh ere.script
> ere.script: syntax error at line 3: `status=$' unexpected
> + fail=1
> + Exit 1
> + set +e
> + exit 1
> + exit 1
> ++ test_dir_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++ cd /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> ++ 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/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $abs_path_dir_ in
> + PATH=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/src:/home/dam/mgar/pkg/.buildsys/v2/gar/bin/sos12-wrappers:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/opt/csw/bin:/opt/csw/bin:/opt/csw/sbin:/opt/csw/sbin:/opt/SUNWspro/bin:/home/dam/mgar/pkg/.buildsys/v2/gar/bin:/usr/bin:/usr/sbin:/usr/java/bin:/usr/ccs/bin:/usr/openwin/bin
> + create_exe_shims_ /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $EXEEXT in
> + return 0
> + shift
> + test 0 '!=' 0
> + export PATH
> + fail=0
> + gawk -f /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/ere.awk /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/ere.tests
> + /bin/sh ere.script
> ere.script: syntax error at line 3: `status=$' unexpected
> + fail=1
> + Exit 1
> + set +e
> + exit 1
> + exit 1
> + remove_tmp_
> + __st=1
> + cleanup_
> + :
> + cd /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> + chmod -R u+rwx /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> + rm -rf /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-ere.PJ8b
> + exit 1

The other is spencer1-locale, but for other reasons this time:

> FAIL: spencer1-locale
> =====================
> 
> ++ initial_cwd_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> ++ fail=0
> +++ testdir_prefix_
> +++ printf gt
> ++ pfx_=gt
> +++ mktempd_ /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests gt-spencer1-locale.XXXX
> +++ case $# in
> +++ destdir_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> +++ template_=gt-spencer1-locale.XXXX
> +++ MAX_TRIES_=4
> +++ case $destdir_ in
> +++ case $template_ in
> ++++ unset TMPDIR
> +++ d=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> +++ case $d in 
> +++ test -d /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> ++++ ls -dgo /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> ++++ tr S -
> +++ perms='drwx------   2       2 Oct 23 09:47 /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P'
> +++ case $perms in
> +++ test 0 = 0
> +++ echo /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> +++ return
> ++ test_dir_=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> ++ cd /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> ++ 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/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $abs_path_dir_ in
> + PATH=/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/src:/home/dam/mgar/pkg/.buildsys/v2/gar/bin/sos12-wrappers:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/bin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/install-isa-sparcv8plus/opt/csw/sbin:/opt/csw/bin:/opt/csw/bin:/opt/csw/sbin:/opt/csw/sbin:/opt/SUNWspro/bin:/home/dam/mgar/pkg/.buildsys/v2/gar/bin:/usr/bin:/usr/sbin:/usr/java/bin:/usr/ccs/bin:/usr/openwin/bin
> + create_exe_shims_ /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/../src
> + case $EXEEXT in
> + return 0
> + shift
> + test 0 '!=' 0
> + export PATH
> + fail=0
> + gawk -v extra_locale= -f /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/spencer1-locale.awk /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/spencer1.tests
> + /bin/sh spencer1-locale.script
> spencer1-locale.script: syntax error at line 2: `status=$' unexpected
> + fail=1
> + Exit 1
> + set +e
> + exit 1
> + exit 1
> + remove_tmp_
> + __st=1
> + cleanup_
> + :
> + cd /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests
> + chmod -R u+rwx /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> + rm -rf /home/dam/mgar/pkg/grep/trunk/work/solaris10-sparc/build-isa-sparcv8plus/grep-2.14.51-7a35/tests/gt-spencer1-locale.fI3P
> + exit 1


Best regards

  -- Dago


-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896

[smime.p7s (application/pkcs7-signature, attachment)]

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Dagobert Michelsen <dam <at> opencsw.org>, 15690 <at> debbugs.gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Wed, 23 Oct 2013 01:14:53 -0700

Thanks for the quick feedback.  Some comments:

Dagobert Michelsen wrote:
> 
> I just tested on Solaris 10 Sparc with Sun Studio 12. First I get
> two warnings:
> 
>   cc: Warning: illegal option -fdiagnostics-show-option
>   cc: Warning: illegal option -funit-at-a-time
> 
> Maybe these should only be added when gcc is detected?

I don't observe these warnings on my Solaris 10 sparc with Sun Studio
12.  (Specifically, this box is patched up through October 15, and is
running Sun C 5.12 SunOS_sparc 2011/11/16.)  I configured with
'./configure CC=cc'.

I suspect you configured with --enable-gcc-warnings.  If so, that's
probably the problem -- that option is intended for use with GCC,
and might not work with Sun C.

> Two tests are failing....
> 
>> FAIL: ere
>> =========
>> ...
>> ere.script: syntax error at line 3: `status=$' unexpected
>> ...
>> FAIL: spencer1-locale
>> =====================
>> ...
>> spencer1-locale.script: syntax error at line 2: `status=$' unexpected

I think these failures are due to the build-tool problem that I mentioned in
<http://lists.gnu.org/archive/html/bug-grep/2013-10/msg00047.html>.
'configure' is selecting /bin/sh, but on Solaris 10 it should
select /bin/bash.  It's then getting confused by the $(...)
syntax, which is not supported by /bin/sh on Solaris 10.

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Wed, 23 Oct 2013 10:20:47 +0200

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

Hi Paul,

Am 23.10.2013 um 10:14 schrieb Paul Eggert <eggert <at> cs.ucla.edu>:
> Thanks for the quick feedback.  Some comments:
> 
> Dagobert Michelsen wrote:
>> 
>> I just tested on Solaris 10 Sparc with Sun Studio 12. First I get
>> two warnings:
>> 
>>  cc: Warning: illegal option -fdiagnostics-show-option
>>  cc: Warning: illegal option -funit-at-a-time
>> 
>> Maybe these should only be added when gcc is detected?
> 
> I don't observe these warnings on my Solaris 10 sparc with Sun Studio
> 12.  (Specifically, this box is patched up through October 15, and is
> running Sun C 5.12 SunOS_sparc 2011/11/16.)  I configured with
> './configure CC=cc'.
> 
> I suspect you configured with --enable-gcc-warnings.  If so, that's
> probably the problem -- that option is intended for use with GCC,
> and might not work with Sun C.

Nope, but we are using git to apply our downstream patches. You may
have a deja vu ;-)
  http://lists.gnu.org/archive/html/bug-gnulib/2013-09/msg00058.html

>> Two tests are failing....
>> 
>>> FAIL: ere
>>> =========
>>> ...
>>> ere.script: syntax error at line 3: `status=$' unexpected
>>> ...
>>> FAIL: spencer1-locale
>>> =====================
>>> ...
>>> spencer1-locale.script: syntax error at line 2: `status=$' unexpected
> 
> I think these failures are due to the build-tool problem that I mentioned in
> <http://lists.gnu.org/archive/html/bug-grep/2013-10/msg00047.html>.
> 'configure' is selecting /bin/sh, but on Solaris 10 it should
> select /bin/bash.  It's then getting confused by the $(...)
> syntax, which is not supported by /bin/sh on Solaris 10.

Yes, exactly.


Best regards

  -- Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896

[smime.p7s (application/pkcs7-signature, attachment)]

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: 15690 <at> debbugs.gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Wed, 23 Oct 2013 11:37:23 -0700

On 10/23/13 01:20, Dagobert Michelsen wrote:
> Nope, but we are using git to apply our downstream patches.

Ah, sorry, I'd forgotten about that.
But this problem should be fixed now, given that we've
applied this patch to grep:

http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=cf70364d5980d76c189ce841b5283f5644a017de

And I can't reproduce the problem on my Solaris 10 sparc
machine (Sun C 5.12 2011/11/16), when I configure this way:

./configure CC=cc --enable-gcc-warnings

Configure outputs a whole bunch of lines like this:

checking whether C compiler handles -Wwrite-strings... no
checking whether C compiler handles -fdiagnostics-show-option... no
checking whether C compiler handles -funit-at-a-time... no

Does it not work that way for you?

There must be something else going on, on your
platform, and I'm not sure what it would be.

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: meyering grep-2.14.51-7a35 fails on Ubuntu 13.10 x86-64
Date: Wed, 23 Oct 2013 17:42:46 -0700

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

On Wed, Oct 23, 2013 at 12:44 AM, Paul Eggert <eggert <at> cs.ucla.edu> wrote:
>>   http://meyering.net/grep/grep-2.14.51-7a35.tar.xz
>
> That snapshot fails "make check" on Ubuntu 13.10 x86-64.
> However, when I generate a tarball from the same git source,
> "make check" works for me.
>
> I think I tracked this down to a discrepancy in the tools used
> to build this snapshot.  But it's getting late so I hope someone
> else can check this.
>
> The snapshot fails because the three tests multibyte-white-space,
> pcre-utf8, surrogate-pair all fail.  And they fail because
> "printf '\x40'" prints "\x40" rather than "@".
> And this happens because the tests are run with
> /bin/sh, not /bin/bash; on Ubuntu /bin/sh is Dash
> and Dash mishandles printf with '\x...'.  And this
> happens because './configure' decides to make SHELL
> /bin/sh rather than /bin/bash.  And I think *this*
> happens because 'configure' was built with Autoconf
> 2.69.112-f181 rather than Autoconf 2.69 (this is
> the Autoconf 2.69 shipped with Fedora 19).  The
> respective 'configure' scripts are pretty hairy,
> though, and I can't easily see why the meyering.net
> snapshot went awry.

Thanks for tracking that down.
I've adjusted the tests not to rely on printf-\xHH.  Here's the patch
I expect to push.
I want to write a replacement printf function (set up via init.sh and
that transforms \xHH to \OOO in the format string), but that will have
to wait until after the release.

[k.txt (text/plain, attachment)]

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <15690 <at> debbugs.gnu.org>
Subject: Re: bug#15690: new snapshot available: grep-2.14.51-7a35
Date: Wed, 23 Oct 2013 22:09:22 -0700

On Tue, Oct 22, 2013 at 11:33 PM, Paul Eggert <eggert <at> cs.ucla.edu> wrote:
> That tarball is built with texinfo 4.13;
> perhaps upgrade to texinfo 5.2?  Among other
> things that will cause the .info files to quote
> 'like this' rather than `like this'.

Thanks for noticing.  I've done that for next time.

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

From: Ineiev <ineiev <at> gnu.org>
To: relevant-mailing-list <at> gnu.org
Cc: platform-testers <at> gnu.org, bug-grep <at> gnu.org
Subject: Re: [platform-testers] new snapshot available: grep-2.14.51-7a35
Date: Thu, 24 Oct 2013 09:39:07 +0400

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

Hello,

On gcc111.fsffrance.org, build fails like
  CCLD     grep
ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_init
ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_destroy
ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_lock
ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_unlock
ld: 0711-345 Use the -bloadmap or -bnoquiet option to obtain more information.

config.log is attached for the reference.

[config.log.gz (application/x-gzip, inline)]

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Ineiev <ineiev <at> gnu.org>, relevant-mailing-list <at> gnu.org
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Thu, 24 Oct 2013 09:58:58 -0700

On 10/23/2013 10:39 PM, Ineiev wrote:
> Hello,
>
> On gcc111.fsffrance.org, build fails like
>   CCLD     grep
> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_init
> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_destroy
> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_lock
> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_unlock
> ld: 0711-345 Use the -bloadmap or -bnoquiet option to obtain more information.
>
> config.log is attached for the reference.

What's the output of 'make V=1'?  Is -lpthread being passed
to the linker?

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

From: Andreas Enge <andreas.enge <at> inria.fr>
To: Jim Meyering <jim <at> meyering.net>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org,
 Paul Eggert <eggert <at> cs.ucla.edu>
Subject: Re: [platform-testers] meyering grep-2.14.51-7a35 fails on Ubuntu
 13.10 x86-64
Date: Thu, 24 Oct 2013 18:49:59 +0200

On Wed, Oct 23, 2013 at 05:42:46PM -0700, Jim Meyering wrote:
> I've adjusted the tests not to rely on printf-\xHH.  Here's the patch
> I expect to push.

Using the patch on the latest snapshot, "make check" succeeds with 

============================================================================
Testsuite summary for GNU grep 2.14.51-7a35
============================================================================
# TOTAL: 152
# PASS:  126
# SKIP:  26
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================

on the following three systems (output of "uname -a"):

Linux debianfu 3.2.0-4-loongson-2f #1 Debian 3.2.51-1 mips64 GNU/Linux
Linux goflexnet 2.6.37.6-goflex #2 PREEMPT Fri May 13 21:06:35 IST 2011 armv5tel GNU/Linux
Linux raspberrypi 3.6.11+ #538 PREEMPT Fri Aug 30 20:42:08 BST 2013 armv6l GNU/Linux

Andreas

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <15690 <at> debbugs.gnu.org>, platform-testers <at> gnu.org
Subject: Re: bug#15690: meyering grep-2.14.51-7a35 fails on Ubuntu 13.10 x86-64
Date: Thu, 24 Oct 2013 11:37:02 -0700

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

I pushed that patch, then tested on a Debian system with /bin/sh == dash.
That exposed at least one more failure (pcre-utf8) due to use of
printf with \xHH, so here's another patch to deal with that and the
few others I found. There is also a tiny clean-up patch.

[k.txt (text/plain, attachment)]

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

From: Jim Meyering <jim <at> meyering.net>
To: Andreas Enge <andreas.enge <at> inria.fr>
Cc: 15690 <15690 <at> debbugs.gnu.org>, platform-testers <at> gnu.org,
 Paul Eggert <eggert <at> cs.ucla.edu>
Subject: Re: [platform-testers] meyering grep-2.14.51-7a35 fails on Ubuntu
 13.10 x86-64
Date: Thu, 24 Oct 2013 11:38:31 -0700

On Thu, Oct 24, 2013 at 9:49 AM, Andreas Enge <andreas.enge <at> inria.fr> wrote:
> On Wed, Oct 23, 2013 at 05:42:46PM -0700, Jim Meyering wrote:
>> I've adjusted the tests not to rely on printf-\xHH.  Here's the patch
>> I expect to push.
>
> Using the patch on the latest snapshot, "make check" succeeds with
>
> ============================================================================
> Testsuite summary for GNU grep 2.14.51-7a35
> ============================================================================
> # TOTAL: 152
> # PASS:  126
> # SKIP:  26
> # XFAIL: 0
> # FAIL:  0
> # XPASS: 0
> # ERROR: 0
> ============================================================================
>
> on the following three systems (output of "uname -a"):
>
> Linux debianfu 3.2.0-4-loongson-2f #1 Debian 3.2.51-1 mips64 GNU/Linux
> Linux goflexnet 2.6.37.6-goflex #2 PREEMPT Fri May 13 21:06:35 IST 2011 armv5tel GNU/Linux
> Linux raspberrypi 3.6.11+ #538 PREEMPT Fri Aug 30 20:42:08 BST 2013 armv6l GNU/Linux

Hi Andreas,
Thanks a lot for testing and reporting the results.

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Thu, 24 Oct 2013 21:42:21 +0200

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

Hi Paul,

Am 23.10.2013 um 20:37 schrieb Paul Eggert <eggert <at> cs.ucla.edu>:
> On 10/23/13 01:20, Dagobert Michelsen wrote:
>> Nope, but we are using git to apply our downstream patches.
> 
> Ah, sorry, I'd forgotten about that.
> But this problem should be fixed now, given that we've
> applied this patch to grep:
> 
> http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=cf70364d5980d76c189ce841b5283f5644a017de
> 
> And I can't reproduce the problem on my Solaris 10 sparc
> machine (Sun C 5.12 2011/11/16), when I configure this way:

This is SolarisStudio 12u3 which is a completely different compiler than the
SunStudio 12 which I am using. TL;DR: marketing does not want version 13.

So we have:
- SunStudio 11:		Sun C 5.8
- SunStudio 12:		Sun C 5.9
- SunStudio 12u1:	Sun C 5.10
- SunStudio 12u2:	Sun C 5.11
- SolarisStudio 12u3:	Sun C 5.12

As you see the "u" versions are new major versions.

> ./configure CC=cc --enable-gcc-warnings
> 
> Configure outputs a whole bunch of lines like this:
> 
> checking whether C compiler handles -Wwrite-strings... no
> checking whether C compiler handles -fdiagnostics-show-option... no
> checking whether C compiler handles -funit-at-a-time... no
> 
> Does it not work that way for you?

No, I too get warnings with --enable-gcc-warnings regardless of the Sun compiler version:

> /opt/solarisstudio12.3/bin/cc -D_STDC_C99= -DHAVE_CONFIG_H -I. -I.. -I.. -I.  -D_REENTRANT -fdiagnostics-show-option -funit-at-a-time  -g -c c-strncasecmp.c
> cc: Warning: Option -fdiagnostics-show-option passed to ld, if ld is invoked, ignored otherwise
> cc: Warning: Option -funit-at-a-time passed to ld, if ld is invoked, ignored otherwise


> There must be something else going on, on your
> platform, and I'm not sure what it would be.


Probably, but I have no idea what it could be.


Best regards

  -- Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896

[smime.p7s (application/pkcs7-signature, attachment)]

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <15690 <at> debbugs.gnu.org>, platform-testers <at> gnu.org
Subject: Re: bug#15690: meyering grep-2.14.51-7a35 fails on Ubuntu 13.10 x86-64
Date: Thu, 24 Oct 2013 13:49:25 -0700

On Thu, Oct 24, 2013 at 11:37 AM, Jim Meyering <jim <at> meyering.net> wrote:
> I pushed that patch, then tested on a Debian system with /bin/sh == dash.
> That exposed at least one more failure (pcre-utf8) due to use of
> printf with \xHH, so here's another patch to deal with that and the
> few others I found.

I noticed that that patch had a harmless extra "I" in one hunk.
I've removed it and pushed.

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

From: Ineiev <ineiev <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 07:51:44 +0400

On 10/24/2013 08:58 PM, Paul Eggert wrote:
> What's the output of 'make V=1'?  Is -lpthread being passed
> to the linker?

        gcc -std=gnu11   -g -O2   -o grep grep.o libgrep.a  ../lib/libgreputils.a -lintl ../lib/libgreputils.a -liconv
ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_init
and so on.

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Ineiev <ineiev <at> gnu.org>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Thu, 24 Oct 2013 22:14:50 -0700

Ineiev wrote:
>         gcc -std=gnu11   -g -O2   -o grep grep.o libgrep.a  ../lib/libgreputils.a -lintl ../lib/libgreputils.a -liconv
> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_init
> and so on.

Yeowch.  This looks like a serious problem on AIX -- grep won't
build at at all.  And it's in the pthreadlib configuration code,
which I don't pretend to understand.  Why does grep care about
pthreadlib at all?

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

From: Ineiev <ineiev <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 09:32:09 +0400

On 10/25/2013 09:14 AM, Paul Eggert wrote:
> Yeowch.  This looks like a serious problem on AIX -- grep won't
> build at at all.

Well, it will, with make LIBS=-lpthread.

>  And it's in the pthreadlib configuration code,
> which I don't pretend to understand.  Why does grep care about
> pthreadlib at all?

Because grep uses it?

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <15690 <at> debbugs.gnu.org>, platform-testers <at> gnu.org,
 Ineiev <ineiev <at> gnu.org>
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Thu, 24 Oct 2013 22:50:51 -0700

On Thu, Oct 24, 2013 at 10:14 PM, Paul Eggert <eggert <at> cs.ucla.edu> wrote:
> Ineiev wrote:
>>         gcc -std=gnu11   -g -O2   -o grep grep.o libgrep.a  ../lib/libgreputils.a -lintl ../lib/libgreputils.a -liconv
>> ld: 0711-317 ERROR: Undefined symbol: .pthread_mutex_init
>> and so on.
>
> Yeowch.  This looks like a serious problem on AIX -- grep won't
> build at at all.  And it's in the pthreadlib configuration code,
> which I don't pretend to understand.  Why does grep care about
> pthreadlib at all?

Yes, that is unfortunate for a package with only single-theaded
programs, like grep.
The "lock" module depends on threadlib, which checks for pthread.h and
may add -pthread.
Several gnulib modules depend on "lock".

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Ineiev <ineiev <at> gnu.org>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 00:02:10 -0700

Ineiev wrote:
> Well, it will, with make LIBS=-lpthread.

'configure' should be doing this automatically.
Can you figure out why it's not?  But in the
meantime perhaps we can suggest './configure LIBS=-lpthread'
as a workaround on AIX, in the release notes or something.

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

From: Ineiev <ineiev <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 13:11:07 +0400

On 10/25/2013 11:02 AM, Paul Eggert wrote:
> Ineiev wrote:
>> Well, it will, with make LIBS=-lpthread.
> 
> 'configure' should be doing this automatically.
> Can you figure out why it's not?

I believe,
--- src/Makefile.am.orig	2013-10-25 12:48:09.000000000 +0400
+++ src/Makefile.am	2013-10-25 12:37:42.000000000 +0400
@@ -21,3 +21,3 @@
 # Tell the linker to omit references to unused shared libraries.
-AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS)
+AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS) $(LIBTHREAD)

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Ineiev <ineiev <at> gnu.org>
Cc: 15690 <at> debbugs.gnu.org, platform-testers <at> gnu.org
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 09:13:17 -0700

Ineiev wrote:

> -AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS)
> +AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS) $(LIBTHREAD)

Thanks!  That explains things.  Though I think it'd be
better to put LIBTHREAD in LDADD, where the other libs go.
I pushed these two patches (the other fixes a minor glitch
with --enable-gcc-warnings introduced by the recent gnulib
update, that I noticed while tracking down the Solaris problem).

From 7c74d12696b4c45670138fad14ef45291032c75e Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert <at> cs.ucla.edu>
Date: Fri, 25 Oct 2013 08:50:53 -0700
Subject: [PATCH 1/2] build: avoid duplicate -funit-at-a-time etc. options

* configure.ac (WERROR_CFLAGS): Don't add -fdiagnostics-show-option
and -funit-at-a-time, as Gnulib does that for us now, and we're
merely piling on duplicats.
---
 configure.ac | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index b1393db..1ebc253 100644
--- a/configure.ac
+++ b/configure.ac
@@ -151,9 +151,6 @@ if test "$gl_gcc_warnings" = yes; then
   # gcc 4.5.0 20090517, and it provokes warnings in cat.c, dd.c, truncate.c
   gl_WARN_ADD([-Wno-logical-op])
 
-  gl_WARN_ADD([-fdiagnostics-show-option])
-  gl_WARN_ADD([-funit-at-a-time])
-
   AC_SUBST([WARN_CFLAGS])
 
   AC_DEFINE([lint], [1], [Define to 1 if the compiler is checking for lint.])
-- 
1.8.3.1


From 25a79f34e9b1485cc0e29f396f4e3b90c0e7c6db Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert <at> cs.ucla.edu>
Date: Fri, 25 Oct 2013 09:09:20 -0700
Subject: [PATCH 2/2] build: port to AIX

Problem reported by Pavel Kharitonov in <http://bugs.gnu.org/15690#68>.
* src/Makefile.am (LDADD): Add $(LIBTHREAD).
---
 src/Makefile.am | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 3d5ca78..4b4c6d1 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -38,7 +38,8 @@ libgrep_a_SOURCES = kwset.c dfa.c searchutils.c dfasearch.c kwsearch.c \
 # replacement functions defined in libgreputils.a.
 LDADD = \
   libgrep.a \
-  ../lib/libgreputils.a $(LIBINTL) ../lib/libgreputils.a $(LIBICONV)
+  ../lib/libgreputils.a $(LIBINTL) ../lib/libgreputils.a $(LIBICONV) \
+  $(LIBTHREAD)
 
 grep_LDADD = $(LDADD) $(LIB_PCRE)
 localedir = $(datadir)/locale
-- 
1.8.3.1

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

From: Jim Meyering <jim <at> meyering.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 15690 <15690 <at> debbugs.gnu.org>, platform-testers <at> gnu.org,
 Ineiev <ineiev <at> gnu.org>
Subject: Re: bug#15690: [platform-testers] new snapshot available:
 grep-2.14.51-7a35
Date: Fri, 25 Oct 2013 11:16:19 -0700

On Fri, Oct 25, 2013 at 9:13 AM, Paul Eggert <eggert <at> cs.ucla.edu> wrote:
> Ineiev wrote:
>
>> -AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS)
>> +AM_LDFLAGS = $(IGNORE_UNUSED_LIBRARIES_CFLAGS) $(LIBTHREAD)
>
> Thanks!  That explains things.  Though I think it'd be
> better to put LIBTHREAD in LDADD, where the other libs go.
> I pushed these two patches (the other fixes a minor glitch
> with --enable-gcc-warnings introduced by the recent gnulib
> update, that I noticed while tracking down the Solaris problem).

Thanks to both of you.  These will be in the upcoming release.

Previous Next

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