Re: [PATCH] Use 'fast-forward' all over the place

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I'll queue your patch with the following interdiff for now.

 * user-manual.txt: do not change emphasis (typography) in this patch---if
   it needs to be changed, that should be a separate patch.

 * builtin-fetch.c: an instance of "non fast-forward" was left
   unconverted; fix it.

 * contrib/hooks/post-receive-email: likewise.

   By the way, in Documentation/git-init.txt there also is another "non
   fast-forward"; since it is not a fix-up to this patch, I left it as-is.
   We may want to change it as well while we are at it.

 * git-gui: this is a straight revert of your patch, so that Shawn can
   apply an undated version of this patch to his tree first, and then I
   can pull the result from him.

   $ git grep -i -n -e 'fast forward' -- git-gui/

   will show what need to be changed.  This is a key string of "mc", so
   msgid in *.po files and git-gui.pot file need to consistently change,
   even though the translated strings in *.po files may stay as before.

If you spot misconversions in here (or other "fast forward" that need to be
converted but I missed), please holler.

Thanks.

---

 Documentation/user-manual.txt    |    2 +-
 builtin-fetch.c                  |    2 +-
 contrib/hooks/post-receive-email |    2 +-
 git-gui/lib/branch_create.tcl    |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt
index 42d92fd..269ec47 100644
--- a/Documentation/user-manual.txt
+++ b/Documentation/user-manual.txt
@@ -2115,7 +2115,7 @@ $ git checkout release && git pull
 
 Important note!  If you have any local changes in these branches, then
 this merge will create a commit object in the history (with no local
-changes git will simply do a fast-forward merge).  Many people dislike
+changes git will simply do a "fast-forward" merge).  Many people dislike
 the "noise" that this creates in the Linux history, so you should avoid
 doing this capriciously in the "release" branch, as these noisy commits
 will become part of the permanent history when you ask Linus to pull
diff --git a/builtin-fetch.c b/builtin-fetch.c
index 89cedc8..6303aa0 100644
--- a/builtin-fetch.c
+++ b/builtin-fetch.c
@@ -287,7 +287,7 @@ static int update_local_ref(struct ref *ref,
 			r ? "unable to update local ref" : "forced update");
 		return r;
 	} else {
-		sprintf(display, "! %-*s %-*s -> %s  (non fast-forward)",
+		sprintf(display, "! %-*s %-*s -> %s  (non-fast-forward)",
 			SUMMARY_WIDTH, "[rejected]", REFCOL_WIDTH, remote,
 			pretty_ref);
 		return 1;
diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email
index 3b075ad..58a35c8 100755
--- a/contrib/hooks/post-receive-email
+++ b/contrib/hooks/post-receive-email
@@ -316,7 +316,7 @@ generate_update_branch_email()
 
 	# List all of the revisions that were removed by this update, in a
 	# fast-forward update, this list will be empty, because rev-list O
-	# ^N is empty.  For a non fast-forward, O ^N is the list of removed
+	# ^N is empty.  For a non-fast-forward, O ^N is the list of removed
 	# revisions
 	fast_forward=""
 	rev=""
diff --git a/git-gui/lib/branch_create.tcl b/git-gui/lib/branch_create.tcl
index f1235c7..3817771 100644
--- a/git-gui/lib/branch_create.tcl
+++ b/git-gui/lib/branch_create.tcl
@@ -77,7 +77,7 @@ constructor dialog {} {
 		-variable @opt_merge
 	pack $w.options.merge.no -side left
 	radiobutton $w.options.merge.ff \
-		-text [mc "Fast-forward Only"] \
+		-text [mc "Fast Forward Only"] \
 		-value ff \
 		-variable @opt_merge
 	pack $w.options.merge.ff -side left


--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]