This is the mail archive of the glibc-cvs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

GNU C Library master sources branch master updated. glibc-2.19-633-gc5c1335


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  c5c13355132e73578bbc0c612ddff964e6199747 (commit)
      from  bd3675f9a3e91edf997c0515f0f1fce1669f038c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=c5c13355132e73578bbc0c612ddff964e6199747

commit c5c13355132e73578bbc0c612ddff964e6199747
Author: Will Newton <will.newton@linaro.org>
Date:   Fri Apr 11 15:21:23 2014 +0100

    test-skeleton.c: Use stdout for error messages
    
    At the moment the test skeleton uses a mixture of stdout and
    stderr for error message output. Using stdout for all test output
    keeps all output correctly ordered and properly redirected to the
    output file. The suggestion to use stdout is also made on the wiki:
    
    https://sourceware.org/glibc/wiki/Testing/Testsuite#Writing_a_test_case
    
    ChangeLog:
    
    2014-06-23  Will Newton  <will.newton@linaro.org>
    
    	* test-skeleton.c (signal_handler): Use printf and %m
    	rather than perror.  Use printf rather than fprintf to
    	stderr.  Use puts rather than fputs to stderr.
    	(main): Likewise.

diff --git a/ChangeLog b/ChangeLog
index 92b567a..11232df 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2014-06-23  Will Newton  <will.newton@linaro.org>
+
+	* test-skeleton.c (signal_handler): Use printf and %m
+	rather than perror.  Use printf rather than fprintf to
+	stderr.  Use puts rather than fputs to stderr.
+	(main): Likewise.
+
 2014-06-22  Ludovic Courtès  <ludo@gnu.org>
 
 	* nscd/nscd.c (thread_info_t): Remove typedef.
diff --git a/test-skeleton.c b/test-skeleton.c
index d7d2f75..286d141 100644
--- a/test-skeleton.c
+++ b/test-skeleton.c
@@ -160,7 +160,7 @@ signal_handler (int sig __attribute__ ((unused)))
     }
   if (killed != 0 && killed != pid)
     {
-      perror ("Failed to kill test process");
+      printf ("Failed to kill test process: %m\n");
       exit (1);
     }
 
@@ -181,16 +181,16 @@ signal_handler (int sig __attribute__ ((unused)))
 #endif
 
   if (killed == 0 || (WIFSIGNALED (status) && WTERMSIG (status) == SIGKILL))
-    fputs ("Timed out: killed the child process\n", stderr);
+    puts ("Timed out: killed the child process");
   else if (WIFSTOPPED (status))
-    fprintf (stderr, "Timed out: the child process was %s\n",
-	     strsignal (WSTOPSIG (status)));
+    printf ("Timed out: the child process was %s\n",
+	    strsignal (WSTOPSIG (status)));
   else if (WIFSIGNALED (status))
-    fprintf (stderr, "Timed out: the child process got signal %s\n",
-	     strsignal (WTERMSIG (status)));
+    printf ("Timed out: the child process got signal %s\n",
+	    strsignal (WTERMSIG (status)));
   else
-    fprintf (stderr, "Timed out: killed the child process but it exited %d\n",
-	     WEXITSTATUS (status));
+    printf ("Timed out: killed the child process but it exited %d\n",
+	    WEXITSTATUS (status));
 
   /* Exit with an error.  */
   exit (1);
@@ -275,7 +275,7 @@ main (int argc, char *argv[])
 
       if (chdir (test_dir) < 0)
 	{
-	  perror ("chdir");
+	  printf ("chdir: %m\n");
 	  exit (1);
 	}
     }
@@ -334,10 +334,10 @@ main (int argc, char *argv[])
 	    data_limit.rlim_cur = MIN ((rlim_t) TEST_DATA_LIMIT,
 				       data_limit.rlim_max);
 	  if (setrlimit (RLIMIT_DATA, &data_limit) < 0)
-	    perror ("setrlimit: RLIMIT_DATA");
+	    printf ("setrlimit: RLIMIT_DATA: %m\n");
 	}
       else
-	perror ("getrlimit: RLIMIT_DATA");
+	printf ("getrlimit: RLIMIT_DATA: %m\n");
 #endif
 
       /* We put the test process in its own pgrp so that if it bogusly
@@ -349,7 +349,7 @@ main (int argc, char *argv[])
     }
   else if (pid < 0)
     {
-      perror ("Cannot fork test program");
+      printf ("Cannot fork test program: %m\n");
       exit (1);
     }
 
@@ -387,18 +387,16 @@ main (int argc, char *argv[])
       if (EXPECTED_SIGNAL != 0)
 	{
 	  if (WTERMSIG (status) == 0)
-	    fprintf (stderr,
-		     "Expected signal '%s' from child, got none\n",
-		     strsignal (EXPECTED_SIGNAL));
+	    printf ("Expected signal '%s' from child, got none\n",
+		    strsignal (EXPECTED_SIGNAL));
 	  else
-	    fprintf (stderr,
-		     "Incorrect signal from child: got `%s', need `%s'\n",
-		     strsignal (WTERMSIG (status)),
-		     strsignal (EXPECTED_SIGNAL));
+	    printf ("Incorrect signal from child: got `%s', need `%s'\n",
+		    strsignal (WTERMSIG (status)),
+		    strsignal (EXPECTED_SIGNAL));
 	}
       else
-	fprintf (stderr, "Didn't expect signal from child: got `%s'\n",
-		 strsignal (WTERMSIG (status)));
+	printf ("Didn't expect signal from child: got `%s'\n",
+		strsignal (WTERMSIG (status)));
       exit (1);
     }
 
@@ -408,8 +406,8 @@ main (int argc, char *argv[])
 #else
   if (WEXITSTATUS (status) != EXPECTED_STATUS)
     {
-      fprintf (stderr, "Expected status %d, got %d\n",
-	       EXPECTED_STATUS, WEXITSTATUS (status));
+      printf ("Expected status %d, got %d\n",
+	      EXPECTED_STATUS, WEXITSTATUS (status));
       exit (1);
     }
 

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog       |    7 +++++++
 test-skeleton.c |   44 +++++++++++++++++++++-----------------------
 2 files changed, 28 insertions(+), 23 deletions(-)


hooks/post-receive
-- 
GNU C Library master sources


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]