From: Eric Blake Date: Fri, 30 Mar 2012 02:47:55 +0000 (-0600) Subject: maint.mk: fix syntax checks without exclusions X-Git-Tag: stable/20120404~4 X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=7f8314c8a7e564f3f0d62bef3cfa68a6bcb9c3d9;p=gnulib.git maint.mk: fix syntax checks without exclusions Commit 727075d0 skipped per-line exclusions if ${exclude} is left undefined, but was immediately broken by commit 44de969c which always initialized the variable to the empty string. * top/maint.mk (_sc_search_regexp): Allow for empty variable. Reported by Daniel P. Berrange. Signed-off-by: Eric Blake (cherry picked from commit 3d0ba3fe7865e7030b4d2097f122e64b16f0a5c9) --- diff --git a/ChangeLog b/ChangeLog index e2209ed26..f110aa51f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ -<<<<<<< HEAD -======= +2012-03-29 Eric Blake + + maint.mk: fix syntax checks without exclusions + * top/maint.mk (_sc_search_regexp): Allow for empty variable. + Reported by Daniel P. Berrange. + 2012-03-24 Bruno Haible gnulib-tool: Avoid unintended error output from 'cmp'. diff --git a/NEWS.stable b/NEWS.stable index a185f33ad..efadc10b8 100644 --- a/NEWS.stable +++ b/NEWS.stable @@ -11,6 +11,7 @@ Date: Sat Mar 24 16:41:21 2012 +0100 with the following additional commits: * [7ce7d6b]->[c07add3] autoupdate + * [3d0ba3f]->[] maint.mk: fix syntax checks without exclusions __NEXTCOMMITMARKER__ ---------------------------------------------------------------------- diff --git a/top/maint.mk b/top/maint.mk index 951b3586f..2228a378b 100644 --- a/top/maint.mk +++ b/top/maint.mk @@ -279,7 +279,7 @@ define _sc_search_regexp if test -n "$$files"; then \ if test -n "$$prohibit"; then \ grep $$with_grep_options $(_ignore_case) -nE "$$prohibit" $$files \ - | grep -vE "$${exclude-^$$}" \ + | grep -vE "$${exclude:-^$$}" \ && { msg="$$halt" $(_sc_say_and_exit) } || :; \ else \ grep $$with_grep_options $(_ignore_case) -LE "$$require" $$files \