Bernhard Voelker wrote: > On 08/31/2012 10:16 AM, Jim Meyering wrote: >> wrote the commit log for you, too. >> Here's the result. I'll wait for your ACK: > > Thanks for wrapping that into a patch. > > I just noticed that there's still a stray > > grep: *.1: No such file or directory
Well spotted. > in the output. That comes from sc_option_desc_uppercase. > What has changed re. the man files? Here's the fix: >From 56661fd87ad1bede1bb84cf3d36d410edd327301 Mon Sep 17 00:00:00 2001 From: Jim Meyering <[email protected]> Date: Fri, 31 Aug 2012 12:08:49 +0200 Subject: [PATCH] build: reenable just-moved/broken syntax-check rule * cfg.mk (sc_option_desc_uppercase): Now that this rule lives in cfg.mk, we must search man/*.1, not "*.1". Reported by Bernhard Voelker. --- cfg.mk | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/cfg.mk b/cfg.mk index 44a43f8..c945ea6 100644 --- a/cfg.mk +++ b/cfg.mk @@ -173,9 +173,8 @@ sc_long_lines: ALL_RECURSIVE_TARGETS += sc_option_desc_uppercase .PHONY: sc_option_desc_uppercase sc_option_desc_uppercase: - @grep '^\\fB\\-' -A1 *.1 | LC_ALL=C grep '\.1.[A-Z][a-z]' && \ - { echo 1>&2 '$@: found initial capitals in --help'; \ - exit 1; } || :; + @grep '^\\fB\\-' -A1 man/*.1 | LC_ALL=C grep '\.1.[A-Z][a-z]' \ + && { echo 1>&2 '$@: found initial capitals in --help'; exit 1; } || : sc_option_desc_uppercase: $(dist_man1_MANS) \ $(patsubst %,man/%.1,$(NO_INSTALL_PROGS_DEFAULT)) \ all_programs -- 1.7.12.146.g16d26b1
