CC-tea/coreutils
1
0
Fork 0
mirror of git://git.sv.gnu.org/coreutils.git synced 2026-04-20 10:51:48 +02:00
Code Issues Projects Releases Wiki Activity

(check-AUTHORS): Don't assume \? works in a sed

Browse Source 
expression; it's not portable.  Problem reported by Albert Chin.
Don't invoke a program more than once.
This commit is contained in:
Paul Eggert
2004-11-03 23:12:55 +00:00
parent fe92cc42b2
commit 90d6db1fbb
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/Makefile.am
View File

@@ -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)