From c5dec32e37e90ce46813efe7dfe8c5500cc55bbc Mon Sep 17 00:00:00 2001 From: "Gary V. Vaughan" Date: Thu, 8 Dec 2011 21:48:03 +0700 Subject: [PATCH] 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 --- cfg.mk | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) 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 -- 2.47.2