diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2004-11-03 23:12:55 +0000 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2004-11-03 23:12:55 +0000 |
| commit | 90d6db1fbb0c6f091b0840f251974536da698640 (patch) | |
| tree | 8fb41161d266a004a31cb09e645f2f342b59a944 /src | |
| parent | fe92cc42b2160ee486b547ff20fde84369a9a65a (diff) | |
| download | coreutils-90d6db1fbb0c6f091b0840f251974536da698640.tar.xz | |
(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.
Diffstat (limited to 'src')
| -rw-r--r-- | src/Makefile.am | 8 |
1 files 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) |