[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new syntax-check rule for @acronym?
From: |
Jim Meyering |
Subject: |
Re: new syntax-check rule for @acronym? |
Date: |
Sat, 27 Mar 2010 07:49:59 +0100 |
address@hidden wrote:
> > This hangs for non-GNU projects, like libvirt, that have no .texi
> > documentation. We really need to make progress on the patch to refactor
> > maint.mk rules per Jose's ideas:
> > http://lists.gnu.org/archive/html/bug-gnulib/2010-02/msg00242.html
> > which would conveniently fix this rule in the process.
>
> IMHO, the address@hidden rule belongs in an optionally-enabled class
> of tests.
>
> Are you thinking about something like
>
> make syntax-check RULESET=gnu-strict
Sort of.
I was thinking of making the optional rules have
the osc_ rule name prefix (optional syntax check), rather than
the usual "sc_" prefix. Then, if a project doesn't want the default,
it would define a variable in cfg.mk to select something different.
> But then, how to define the sets of rules?
Let the form of the rule name define the sets.
Currently it's done like this with the "sc_" prefix:
_cfg_mk := $(shell test -f $(srcdir)/cfg.mk && echo '$(srcdir)/cfg.mk')
# Collect the names of rules starting with `sc_'.
syntax-check-rules := $(sort $(shell sed -n
's/^\(sc_[a-zA-Z0-9_-]*\):.*/\1/p' \
$(srcdir)/$(ME) $(_cfg_mk)))
.PHONY: $(syntax-check-rules)