From: Derek Price <derek@ximbiot.com>
Date: Tue, 29 Apr 2003 20:54:12 +0000 (+0000)
Subject: * lib/autotest/general.m4 (AT_KEYWORDS): Don't use a comma as the
X-Git-Tag: AUTOCONF-2.57a~51
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=044418223e352ff74a178c486c45fccf66b90bc5;p=thirdparty%2Fautoconf.git

* lib/autotest/general.m4 (AT_KEYWORDS): Don't use a comma as the
seperator with m4_append_uniq().  It doesn't work.
(AT_CLEANUP): Add `;' to end of at_help_all.
(AT_INIT): Allow --keywords to be specified more than once.  When
grepping $at_help_all for keywords, use the field and keyword
seperators to ensure a complete keyword match.  Alter at_prev handling
to support the new --keywords behavior.
---

diff --git a/ChangeLog b/ChangeLog
index c04aba161..3242b8ad0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2003-04-29  Derek Price  <derek@ximbiot.com>
+
+	* lib/autotest/general.m4 (AT_KEYWORDS): Don't use a comma as the
+	seperator with m4_append_uniq().  It doesn't work.
+	(AT_CLEANUP): Add `;' to end of at_help_all.
+	(AT_INIT): Allow --keywords to be specified more than once.  When
+	grepping $at_help_all for keywords, use the field and keyword
+	seperators to ensure a complete keyword match.  Alter at_prev handling
+	to support the new --keywords behavior.
+
 2003-04-27  Karl Berry  <karl@freefriends.org>
 
 	* doc/autoconf.texi: Make the dir entries in the autoconf manual
diff --git a/lib/autotest/general.m4 b/lib/autotest/general.m4
index 1e712387d..57612c1a8 100644
--- a/lib/autotest/general.m4
+++ b/lib/autotest/general.m4
@@ -214,9 +214,8 @@ for at_option
 do
   # If the previous option needs an argument, assign it.
   if test -n "$at_prev"; then
-    eval "$at_prev=\$at_option"
+    at_option=$at_prev=$at_option
     at_prev=
-    continue
   fi
 
   at_optarg=`expr "x$at_option" : 'x[[^=]]*=\(.*\)'`
@@ -295,10 +294,10 @@ do
 
     # Keywords.
     --keywords | -k )
-	at_prev=at_keywords
+	at_prev=--keywords
 	;;
     --keywords=* )
-	at_keywords=$at_optarg
+	at_keywords="$at_keywords,$at_optarg"
 	;;
 m4_divert_pop([PARSE_ARGS])dnl
 dnl Process *=* last to allow for user specified --option=* type arguments.
@@ -330,7 +329,7 @@ if test -n "$at_keywords"; then
   do
     # It is on purpose that we match the test group titles too.
     at_groups_selected=`echo "$at_groups_selected" |
-			grep -i "^[[^;]]*;[[^;]]*;.*$at_keyword"`
+			grep -i "^[[^;]]*;[[^;]]*.*[[; ]]$at_keyword[[ ;]]"`
   done
   at_groups_selected=`echo "$at_groups_selected" | sed 's/;.*//'`
   # Smash the end of lines.
@@ -802,7 +801,7 @@ m4_divert_push([TESTS])dnl
 # ---------------------
 # Declare a list of keywords associated to the current test group.
 m4_define([AT_KEYWORDS],
-[m4_append_uniq([AT_keywords], [$1], [,])])
+[m4_append_uniq([AT_keywords], [$1], [ ])])
 
 
 # AT_CLEANUP
@@ -810,7 +809,7 @@ m4_define([AT_KEYWORDS],
 # Complete a group of related tests.
 m4_define([AT_CLEANUP],
 [m4_append([AT_help],
-at_help_all=$at_help_all'm4_defn([AT_ordinal]);m4_defn([AT_line]);m4_defn([AT_description]);m4_ifdef([AT_keywords], [m4_defn([AT_keywords])])
+at_help_all=$at_help_all'm4_defn([AT_ordinal]);m4_defn([AT_line]);m4_defn([AT_description]);m4_ifdef([AT_keywords], [m4_defn([AT_keywords])]);
 '
 )dnl
     $at_times_skip || times >$at_times_file