branch-1_4 show why tests are skipped

Eric Blake <[email protected]>
Newsgroups gmane.comp.gnu.m4.patches
Message-ID <[email protected]>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Based on a suggestion from Bruno on bug-coreutils for the hello project,
the branch now shows why tests are skipped.  This does not need to be
ported to head, since autotest provides a different framework for skipped
tests (and it is easier to inspect testsuite.log to see why a test is
skipped).

2006-08-22  Eric Blake  <[email protected]>

	* doc/m4.texinfo: Minor improvements.
	(Changeword, Sysval, Maketemp):  State why a test is skipped.
	* checks/check-them: Likewise.

- --
Life is short - so eat dessert first!

Eric Blake             [email protected]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.1 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFE62Dz84KuGfSFAYARAn6WAJ9sZqpMbPam5RFm+Mwr+TYSN9aYUQCg2Rj9
7hOiMKPkDAUKKVHh4La4Om0=
=dG+p
-----END PGP SIGNATURE-----

_______________________________________________
M4-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/m4-patches
m4.patch127 (text/plain, 4 KB)
Index: checks/check-them
===================================================================
RCS file: /sources/m4/m4/checks/Attic/check-them,v
retrieving revision 1.1.1.1.2.10
diff -u -p -r1.1.1.1.2.10 check-them
--- checks/check-them	4 Aug 2006 13:19:41 -0000	1.1.1.1.2.10
+++ checks/check-them	22 Aug 2006 19:52:34 -0000
@@ -51,6 +51,7 @@ do
 
   if test $? = 77 ; then
     skipped="$skipped $file"
+    cat $err
     continue
   fi
 
Index: doc/m4.texinfo
===================================================================
RCS file: /sources/m4/m4/doc/m4.texinfo,v
retrieving revision 1.1.1.1.2.68
diff -u -p -r1.1.1.1.2.68 m4.texinfo
--- doc/m4.texinfo	18 Aug 2006 03:39:11 -0000	1.1.1.1.2.68
+++ doc/m4.texinfo	22 Aug 2006 19:52:34 -0000
@@ -706,9 +706,9 @@ is not a part of either a name or a stri
 contain comments.  @acronym{GNU} @code{m4} does not yet understand
 locales; all operations are byte-oriented rather than
 character-oriented.  However, @code{m4} is eight-bit clean, so you can
-use non-ASCII characters in quoted strings (@pxref{Changequote}),
+use non-@sc{ascii} characters in quoted strings (@pxref{Changequote}),
 comments (@pxref{Changecom}), and macro names (@pxref{Indir}), with the
-exception of the NUL character (the zero byte).
+exception of the @sc{nul} character (the zero byte @samp{'\0'}).
 
 @menu
 * Names::                       Macro names
@@ -723,9 +723,10 @@ exception of the NUL character (the zero
 
 @cindex names
 A name is any sequence of letters, digits, and the character @kbd{_}
-(underscore), where the first character is not a digit.  If a
-name has a macro definition, it will be subject to macro expansion
-(@pxref{Macros}).
+(underscore), where the first character is not a digit.  @code{m4} will
+use the longest such sequence found in the input.  If a name has a
+macro definition, it will be subject to macro expansion
+(@pxref{Macros}).  Names are case-sensitive.
 
 Examples of legal names are: @samp{foo}, @samp{_tmp}, and @samp{name01}.
 
@@ -2670,8 +2671,8 @@ Relaxing the lexical rules of @code{m4} 
 you wanted to apply translations to a file of numbers:
 
 @example
-ifdef(`changeword', `', `m4exit(`77')')
-@result{}
+ifdef(`changeword', `', `errprint(` skipping: no changeword support
+')m4exit(`77')')dnl
 changeword(`[_a-zA-Z0-9]+')
 @result{}
 define(`1', `0')1
@@ -2683,8 +2684,8 @@ make some of the builtins unavailable.  
 accidental call of builtins, for example:
 
 @example
-ifdef(`changeword', `', `m4exit(`77')')
-@result{}
+ifdef(`changeword', `', `errprint(` skipping: no changeword support
+')m4exit(`77')')dnl
 define(`_indir', defn(`indir'))
 @result{}
 changeword(`_[_a-zA-Z0-9]*')
@@ -2706,8 +2707,8 @@ supplied contains any grouped subexpress
 the first of these is discarded before symbol lookup.  So:
 
 @example
-ifdef(`changeword', `', `m4exit(`77')')
-@result{}
+ifdef(`changeword', `', `errprint(` skipping: no changeword support
+')m4exit(`77')')dnl
 changecom(`/*', `*/')dnl
 define(`foo', `bar')dnl
 changeword(`#\([_a-zA-Z0-9]*\)')
@@ -2740,8 +2741,8 @@ relief.  For example, here is the same i
 Then, the @code{m4} version:
 
 @example
-ifdef(`changeword', `', `m4exit(`77')')
-@result{}
+ifdef(`changeword', `', `errprint(` skipping: no changeword support
+')m4exit(`77')')dnl
 define(`a', `errprint(`Hello')')dnl
 changeword(`@@\([_a-zA-Z0-9]*\)')
 @result{}
@@ -4019,7 +4020,8 @@ signal number shifted left by eight bits
 @example
 dnl This test assumes kill is a shell builtin, and that signals are
 dnl recognizable.
-ifdef(`__unix__', , `m4exit(`77')')dnl
+ifdef(`__unix__', , `errprint(` skipping: syscmd does not have unix semantics
+')m4exit(`77')')dnl
 syscmd(`kill -13 $$')
 @result{}
 sysval
@@ -4066,7 +4068,8 @@ maketemp(`/tmp/fooXXXXXX')
 @example
 dnl This test assumes /tmp is a valid directory name, which is not true
 dnl for native Windows.
-ifdef(`__unix__', , `m4exit(`77')')dnl
+ifdef(`__unix__', , `errprint(` skipping: not sure /tmp exists
+')m4exit(`77')')dnl
 define(`file1', maketemp(`/tmp/fooXXXXXX'))dnl
 define(`file2', maketemp(`/tmp/fooXXXXXX'))dnl
 ifelse(file1, file2, `same', `different')
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.