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 dc89660c7c62744607578ae77e30d3f5164664f4..ef1637ad04759167d504eff612c19b454ed2f3c6 100644 (file)
--- 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