From: Gary V. Vaughan <gary@gnu.org>
Date: Wed, 9 Jun 2010 12:34:34 +0000 (+0700)
Subject: Fix a quoting bug with ./commit -a 'A U Thor'.
X-Git-Tag: v2.2.10~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c39115a3f83ceceb31d7ebba4af9b65bde6de737;p=thirdparty%2Flibtool.git

Fix a quoting bug with ./commit -a 'A U Thor'.

* clcommit.m4sh (func_commit): Use func_show_eval when calling
git, so that the previously func_quote_for_eval()ed $git_flags is
expanded properly.

Signed-off-by: Gary V. Vaughan <gary@gnu.org>
---

diff --git a/ChangeLog b/ChangeLog
index 054399949..a1bc4653a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-06-09  Gary V. Vaughan  <gary@gnu.org>
+
+	Fix a quoting bug with ./commit -a 'A U Thor'.
+	* clcommit.m4sh (func_commit): Use func_show_eval when calling
+	git, so that the previously func_quote_for_eval()ed $git_flags is
+	expanded properly.
+
 2009-09-15  Peter Rosin  <peda@lysator.liu.se>
 
 	Don't try the MSYS libdir path on MinGW.
diff --git a/clcommit.m4sh b/clcommit.m4sh
index 93d5f9924..6157cd295 100644
--- a/clcommit.m4sh
+++ b/clcommit.m4sh
@@ -266,18 +266,11 @@ func_commit ()
     test $# -gt 0 && subject="$@"
 
     test $# -gt 0 || { set dummy -a; shift; }
-    func_verbose "$GIT commit $git_flags -F $log_file ${1+$@}"
-    $opt_dry_run ||
-        $GIT commit $git_flags -F $log_file ${1+"$@"} || exit $EXIT_FAILURE
+    func_show_eval "$GIT commit $git_flags -F $log_file ${1+$@}" "exit $EXIT_FAILURE"
 
     $opt_push && {
-      func_verbose "$GIT push"
-      $opt_dry_run || $GIT push
-
-      $opt_tags && {
-	  func_verbose "$GIT push --tags"
-	  $opt_dry_run || $GIT push --tags
-      }
+      func_show_eval "$GIT push"
+      $opt_tags && func_show_eval "$GIT push --tags"
     }
 
     :