bug-gnulib
[Top][All Lists]
Advanced

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]