Re: new syntax-check rule for @acronym?

[Simon Josefsson]

address@hidden (Karl Berry) writes:

>     syntax-check test for this seems useful, to foster harmonization across
>     GNU packages.  How about the patch below?
>
> I have no objection, certainly.

I pushed the patch below.

Thanks,
Simon

>From 59255e746536e35c3ebd748535cc0469720949e9 Mon Sep 17 00:00:00 2001
From: Simon Josefsson <address@hidden>
Date: Wed, 24 Mar 2010 08:45:04 +0100
Subject: [PATCH] top/maint.mk (sc_texinfo_acronym): Add rule.

Based on suggestions from address@hidden (Karl Berry).
---
 ChangeLog    |    5 +++++
 top/maint.mk |    7 +++++++
 2 files changed, 12 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 8e31e9a..8166c5d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2010-03-24  Simon Josefsson  <address@hidden>
+
+       * top/maint.mk (sc_texinfo_acronym): Add rule, based on
+       suggestions from address@hidden (Karl Berry).
+
 2010-03-22  Eric Blake  <address@hidden>
 
        gethostname: further tweaks
diff --git a/top/maint.mk b/top/maint.mk
index d2248d8..67421f2 100644
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -561,6 +561,13 @@ sc_GFDL_version:
        @re='$(_GFDL_regexp)' msg='GFDL vN, N!=3'                       \
          $(_prohibit_regexp)
 
+# Don't use Texinfo @acronym{} as it is not a good idea.
+sc_texinfo_acronym:
+       @grep -nE '@acronym{'                                           \
+           $$($(VC_LIST_EXCEPT) | grep -E '\.texi$$') &&               \
+         { echo '$(ME): found use of Texinfo @acronym{}' 1>&2;         \
+           exit 1; } || :
+
 cvs_keywords = \
   Author|Date|Header|Id|Name|Locker|Log|RCSfile|Revision|Source|State
 
-- 
1.7.0.3