Re: [platform-testers] new snapshot available: diffutils-3.2.37-f0b0

[Paul Eggert]

Thanks for the testing report.

I installed the following changes to gnulib, and they should propagate
to diffutils in due course.  The one to test-sh-quote
should fix your problem with lintl.  The one to test-raise.c
is a porting gotcha that may be independent of Solaris, but might
as well fix it now.

Unfortunately the stack traces that you sent do not provide enough
info for me to see what the problem is on Solaris 9 with test-raise
and with test-sigaction.  Perhaps you can try again, compiling with
-g and without -O, and get more info that way.  You can run
'./configure CFLAGS=-g; make clean; make; make check' to do that.

>From ced5dc5cac2827d80a02bc9f9015ad388ef0428a Mon Sep 17 00:00:00 2001
From: Paul Eggert <address@hidden>
Date: Tue, 30 Oct 2012 11:27:56 -0700
Subject: [PATCH 1/2] sh-quote-tests: port to Solaris 9

* modules/sh-quote-tests (test_sh_quote_LDADD): Add @address@hidden
Problem reported by Dagobert Michelsen in
<http://lists.gnu.org/archive/html/bug-gnulib/2012-10/msg00114.html>.
---
 ChangeLog              | 7 +++++++
 modules/sh-quote-tests | 1 +
 2 files changed, 8 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index ea23e5a..0ab006c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2012-10-30  Paul Eggert  <address@hidden>
+
+       sh-quote-tests: port to Solaris 9
+       * modules/sh-quote-tests (test_sh_quote_LDADD): Add @address@hidden
+       Problem reported by Dagobert Michelsen in
+       <http://lists.gnu.org/archive/html/bug-gnulib/2012-10/msg00114.html>.
+
 2012-10-28  Jim Meyering  <address@hidden>
 
        maint.mk: rename a new configurable variable
diff --git a/modules/sh-quote-tests b/modules/sh-quote-tests
index c4a6286..ac6101a 100644
--- a/modules/sh-quote-tests
+++ b/modules/sh-quote-tests
@@ -8,3 +8,4 @@ configure.ac:
 Makefile.am:
 TESTS += test-sh-quote
 check_PROGRAMS += test-sh-quote
+test_sh_quote_LDADD = $(LDADD) @LIBINTL@
-- 
1.7.11.7


>From 381db26e9661594748d44b8dba7edfd9d8b2c3d2 Mon Sep 17 00:00:00 2001
From: Paul Eggert <address@hidden>
Date: Tue, 30 Oct 2012 11:29:01 -0700
Subject: [PATCH 2/2] test-raise: don't assume 199 is an invalid signal

* tests/test-raise.c (main): Don't assume 199 is not a signal number.
---
 ChangeLog          | 3 +++
 tests/test-raise.c | 1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 0ab006c..bf77930 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2012-10-30  Paul Eggert  <address@hidden>
 
+       test-raise: don't assume 199 is an invalid signal
+       * tests/test-raise.c (main): Don't assume 199 is not a signal number.
+
        sh-quote-tests: port to Solaris 9
        * modules/sh-quote-tests (test_sh_quote_LDADD): Add @address@hidden
        Problem reported by Dagobert Michelsen in
diff --git a/tests/test-raise.c b/tests/test-raise.c
index 9ec0781..e7cfcb9 100644
--- a/tests/test-raise.c
+++ b/tests/test-raise.c
@@ -39,7 +39,6 @@ main (void)
 {
   /* Test behaviour for invalid argument.  */
   ASSERT (raise (-1) != 0);
-  ASSERT (raise (199) != 0);
 
   /* Test behaviour for SIGINT.  */
   ASSERT (signal (SIGINT, handler) != SIG_ERR);
-- 
1.7.11.7