From a159d594facba612e60d58539eeba80e2a0d3413 Mon Sep 17 00:00:00 2001
From: Stefano Lattarini <stefano.lattarini@gmail.com>
Date: Thu, 30 Aug 2012 18:19:53 +0200
Subject: build: some refactorings in cfg.mk

Partly a minor cleanup, partly a preparation for future changes.

* Makefile.am (all_programs): Rename ...
(all-progs-but-lbracket): ... like this, and re-define it to expand
at make time rather than only at recipe time (i.e., using $(shell ...)
instead of `...`).
(check-programs-vs-x): Adjust.
---
 cfg.mk | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'cfg.mk')

diff --git a/cfg.mk b/cfg.mk
index b80a61f63..92bbadc24 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -206,16 +206,17 @@ check-x-vs-1:
 	  | $(ASSORT) -u | diff - $$t || { rm $$t; exit 1; };		\
 	rm $$t
 
-all_programs =								\
-      (cd ./src && MAKEFLAGS= $(MAKE) -s all_programs.list)		\
-       | grep -v '\['
+# Writing a portable rule to generate a manpage like '[.1' would be
+# a nightmare.
+all-progs-but-lbracket = $(filter-out [, $(shell \
+      (cd ./src && MAKEFLAGS= ${MAKE} -s all_programs.list)))
 
 # Ensure that for each .x file in the 'man/' subdirectory, there is a
 # corresponding coreutils program.
 .PHONY: check-programs-vs-x
 check-programs-vs-x: all_programs
 	@status=0;					\
-	for p in dummy `$(all_programs)`; do		\
+	for p in dummy $(all-progs-but-lbracket); do	\
 	  test $$p = dummy && continue;			\
 	  test $$p = ginstall && p=install || : ;	\
 	  test -f $(srcdir)/man/$$p.x			\
-- 
cgit v1.2.3-54-g00ecf