From: Jim Meyering Date: Thu, 2 Jan 2014 04:25:44 +0000 (-0800) Subject: maint: fix public-submodule-commit to work with newer git X-Git-Tag: v1.0~7468 X-Git-Url: https://gitweb.git.savannah.gnu.org/gitweb/?a=commitdiff_plain;h=7a0c729aacc3716a665ce3b0c8cb34843bdecc75;p=gnulib.git maint: fix public-submodule-commit to work with newer git * top/maint.mk (public-submodule-commit): Remove excess quoting. We were over-quoting the test arguments, and somewhere prior to version 1.8.5.2.229, git stopped removing those excess quotes, which made the test fail, since the unexpanded strings would always differ; using GIT_TRACE=1 confirmed that the git merge-base command wasn't even being run. --- diff --git a/ChangeLog b/ChangeLog index 4335802c42..c3f5dc145a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +2014-01-01 Jim Meyering + + maint: fix public-submodule-commit to work with newer git + * top/maint.mk (public-submodule-commit): Remove excess quoting. + We were over-quoting the test arguments, and somewhere prior to + version 1.8.5.2.229, git stopped removing those excess quotes, + which made the test fail, since the unexpanded strings would + always differ; using GIT_TRACE=1 confirmed that the git merge-base + command wasn't even being run. + 2014-01-01 Paul Eggert doc: update main copyright year diff --git a/top/maint.mk b/top/maint.mk index 7420af5fdb..cae2dda039 100644 --- a/top/maint.mk +++ b/top/maint.mk @@ -1395,8 +1395,8 @@ public-submodule-commit: && git --version >/dev/null 2>&1; then \ cd $(srcdir) && \ git submodule --quiet foreach \ - test '"$$(git rev-parse "$$sha1")"' \ - = '"$$(git merge-base origin "$$sha1")"' \ + 'test "$$(git rev-parse "$$sha1")" \ + = "$$(git merge-base origin "$$sha1")"' \ || { echo '$(ME): found non-public submodule commit' >&2; \ exit 1; }; \ else \