From ab305ad5ca504748d78bbb0e8d398781c6be940b Mon Sep 17 00:00:00 2001 From: Andrea Bolognani Date: Mon, 23 May 2022 14:41:08 +0200 Subject: [PATCH] syntax-check: Simplify VC_LIST_ALWAYS_EXCLUDE_REGEX Most of the pattern is no longer relevant, because the files it was intended to match have been dropped from the repository. Specifically: files commit date ------------------ ------------ ---------- *.gif 6cb131e5cbd0 2022-01-19 *.fig 9ad637c9651f 2020-07-10 docs/news*.html.in f45735786a3d 2020-06-02 docs/*.patch 6be034a8c062 2018-08-23 We can also avoid having a fallback value for the pattern: that made sense when the implementation was coming from gnulib, as they wouldn't be able to know in advance if the user would need to provide their own exclude patterns, but that scenario is no longer relevant to us. Signed-off-by: Andrea Bolognani Reviewed-by: Michal Privoznik --- build-aux/syntax-check.mk | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk index d6f2a0602d..1252cb8eb4 100644 --- a/build-aux/syntax-check.mk +++ b/build-aux/syntax-check.mk @@ -40,10 +40,6 @@ _equal = $(and $(findstring $(1),$(2)),$(findstring $(2),$(1))) VC_LIST = cd $(top_srcdir); git ls-tree -r 'HEAD:' | \ sed -n "s|^100[^ ]*.||p" -# You can override this variable in syntax-check.mk to set your own regexp -# matching files to ignore. -VC_LIST_ALWAYS_EXCLUDE_REGEX ?= ^$$ - # This is to preprocess robustly the output of $(VC_LIST), so that even # when $(top_srcdir) is a pathological name like "....", the leading sed command # removes only the intended prefix. @@ -110,7 +106,7 @@ syntax-check: $(local-check) # Files that should never cause syntax check failures. VC_LIST_ALWAYS_EXCLUDE_REGEX = \ - (^(docs/(news(-[0-9]*)?\.html\.in|.*\.patch))|\.(po|fig|gif|ico|png))$$ + \.(po|ico|png)$$ # Avoid uses of write(2). Either switch to streams (fwrite), or use # the safewrite wrapper.