From 90d6db1fbb0c6f091b0840f251974536da698640 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Wed, 3 Nov 2004 23:12:55 +0000
Subject: (check-AUTHORS): Don't assume \? works in a sed expression; it's not
 portable.  Problem reported by Albert Chin. Don't invoke a program more than
 once.

---
 src/Makefile.am | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 329569799..d1c039f0c 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -248,7 +248,7 @@ au_actual = authors-actual
 .PHONY: check-AUTHORS
 check-AUTHORS: $(all_programs)
 	rm -f $(au_actual) $(au_dotdot)
-	for i in $(all_programs); do			\
+	for i in `ls $(all_programs) | $(ASSORT) -u`; do \
 	  test "$$i" = '[' && continue;			\
 	  exe=$$i;					\
 	  if test "$$i" = install; then			\
@@ -256,11 +256,9 @@ check-AUTHORS: $(all_programs)
 	  elif test "$$i" = test; then			\
 	    exe='[';					\
 	  fi;						\
-	  ./printf "$$i: ";				\
 	  ./$$exe --version				\
-	    |sed -n '/Written by /s///p'		\
-	    |sed 's/,\? and /, /;s/\.$$//';		\
-	done | $(ASSORT) > $(au_actual)
+	    |sed -n '/Written by /{ s//'"$$i"': /; s/,* and /, /; s/\.$$//; p; }'; \
+	done > $(au_actual)
 	sed -n '/:/p' $(top_srcdir)/AUTHORS > $(au_dotdot)
 	diff $(au_actual) $(au_dotdot) && rm -f $(au_actual) $(au_dotdot)
 
-- 
cgit v1.2.3-70-g09d2