From: Gary V. Vaughan <gary@gnu.org>
Date: Thu, 8 Dec 2011 14:48:03 +0000 (+0700)
Subject: maint: disable prohibit_always-defined_macros syntax check.
X-Git-Tag: v2.4.2.418~205
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0009ef898866ec536c04a1d20b6f01947e07070b;p=thirdparty%2Flibtool.git

maint: disable prohibit_always-defined_macros syntax check.

Aside from Apple grep silently failing to run this test and
report violations, libtool still uses non-gnulib versions of
dirent.h and argz.h which use the same macros, but are not
always-defined.
* cfg.mk (local-checks-to-fix): Move
sc_prohibit_always-defined_macros from here...
(local-checks-to-skip): ...to here.
Reported by Peter O'Gorman.

Signed-off-by: Gary V. Vaughan <gary@gnu.org>
---

diff --git a/cfg.mk b/cfg.mk
index dc89660c7..ef1637ad0 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -38,7 +38,6 @@ endif
 VC_LIST_ALWAYS_EXCLUDE_REGEX = ^mail/
 
 local-checks-to-fix =				\
-	sc_prohibit_always-defined_macros	\
 	sc_require_config_h			\
 	sc_require_config_h_first
 
@@ -46,12 +45,16 @@ local-checks-to-skip =				\
 	$(local-checks-to-fix)			\
 	sc_GPL_version				\
 	sc_cast_of_x_alloc_return_value		\
+	sc_prohibit_always-defined_macros	\
 	sc_trailing_blank			\
 	sc_unmarked_diagnostics
 
 # GPL_version: checks for GPLv3, which we don't use
 # cast_of_x_alloc_return_value:
 #         We support C++ compilation which does require casting here.
+# prohibit_always-defined_macros:
+#	we have our own argz and dirent, which use the same macros but
+#	which are not always-defined in our case.
 # trailing_blank: flags valid rfc3676 separators
 # unmarked_diagnostics: libtool isn't internationalized