{"thread":{"id":"21342","subject":"[PATCH] Use fast-forward","startedAt":"2009-10-24T08:31:31Z","lastAt":"2009-10-25T09:06:16Z","messageCount":14,"participants":["Felipe Contreras","Nanako Shiraishi","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"125827","messageId":"1256373092-15126-1-git-send-email-felipe.contreras@gmail.com","threadId":"21342","inReplyTo":null,"subject":"[PATCH] Use fast-forward","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-24T08:31:31Z","receivedAt":"2009-10-24T08:31:31Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"As suggested in the mailing list, now I've replaced all instances of 'fast\nforward' with 'fast-forward'.\n\nFelipe Contreras (1):\n  Use 'fast-forward' all over the place\n\n Documentation/config.txt                     |    2 +-\n Documentation/git-http-push.txt              |    4 ++--\n Documentation/git-push.txt                   |   14 +++++++-------\n Documentation/git-read-tree.txt              |    2 +-\n Documentation/git-receive-pack.txt           |    2 +-\n Documentation/git-reset.txt                  |    4 ++--\n Documentation/git-send-pack.txt              |    4 ++--\n Documentation/gitcore-tutorial.txt           |    4 ++--\n Documentation/githooks.txt                   |    2 +-\n Documentation/glossary-content.txt           |    4 ++--\n Documentation/howto/maintain-git.txt         |    2 +-\n Documentation/howto/revert-branch-rebase.txt |    4 ++--\n Documentation/howto/update-hook-example.txt  |    2 +-\n Documentation/pull-fetch-param.txt           |    4 ++--\n Documentation/user-manual.txt                |   14 +++++++-------\n builtin-fetch--tool.c                        |    8 ++++----\n builtin-fetch.c                              |    4 ++--\n builtin-merge.c                              |    8 ++++----\n builtin-push.c                               |    2 +-\n builtin-receive-pack.c                       |    4 ++--\n builtin-remote.c                             |    2 +-\n builtin-send-pack.c                          |    2 +-\n contrib/examples/git-merge.sh                |    8 ++++----\n contrib/examples/git-resolve.sh              |    2 +-\n contrib/hooks/post-receive-email             |    6 +++---\n git-gui/lib/branch_create.tcl                |    2 +-\n git-merge-octopus.sh                         |    2 +-\n git-pull.sh                                  |    2 +-\n git-rebase--interactive.sh                   |    6 +++---\n git-rebase.sh                                |    2 +-\n t/t1001-read-tree-m-2way.sh                  |    6 +++---\n t/t5505-remote.sh                            |    2 +-\n t/t5518-fetch-exit-status.sh                 |    2 +-\n t/t6028-merge-up-to-date.sh                  |    2 +-\n transport.c                                  |    2 +-\n unpack-trees.c                               |    2 +-\n 36 files changed, 72 insertions(+), 72 deletions(-)\n"},{"id":"125828","messageId":"1256373092-15126-2-git-send-email-felipe.contreras@gmail.com","threadId":"21342","inReplyTo":"1256373092-15126-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH] Use 'fast-forward' all over the place","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-24T08:31:32Z","receivedAt":"2009-10-24T08:31:32Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"It's a compound word.\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n Documentation/config.txt                     |    2 +-\n Documentation/git-http-push.txt              |    4 ++--\n Documentation/git-push.txt                   |   14 +++++++-------\n Documentation/git-read-tree.txt              |    2 +-\n Documentation/git-receive-pack.txt           |    2 +-\n Documentation/git-reset.txt                  |    4 ++--\n Documentation/git-send-pack.txt              |    4 ++--\n Documentation/gitcore-tutorial.txt           |    4 ++--\n Documentation/githooks.txt                   |    2 +-\n Documentation/glossary-content.txt           |    4 ++--\n Documentation/howto/maintain-git.txt         |    2 +-\n Documentation/howto/revert-branch-rebase.txt |    4 ++--\n Documentation/howto/update-hook-example.txt  |    2 +-\n Documentation/pull-fetch-param.txt           |    4 ++--\n Documentation/user-manual.txt                |   14 +++++++-------\n builtin-fetch--tool.c                        |    8 ++++----\n builtin-fetch.c                              |    4 ++--\n builtin-merge.c                              |    8 ++++----\n builtin-push.c                               |    2 +-\n builtin-receive-pack.c                       |    4 ++--\n builtin-remote.c                             |    2 +-\n builtin-send-pack.c                          |    2 +-\n contrib/examples/git-merge.sh                |    8 ++++----\n contrib/examples/git-resolve.sh              |    2 +-\n contrib/hooks/post-receive-email             |    6 +++---\n git-gui/lib/branch_create.tcl                |    2 +-\n git-merge-octopus.sh                         |    2 +-\n git-pull.sh                                  |    2 +-\n git-rebase--interactive.sh                   |    6 +++---\n git-rebase.sh                                |    2 +-\n t/t1001-read-tree-m-2way.sh                  |    6 +++---\n t/t5505-remote.sh                            |    2 +-\n t/t5518-fetch-exit-status.sh                 |    2 +-\n t/t6028-merge-up-to-date.sh                  |    2 +-\n transport.c                                  |    2 +-\n unpack-trees.c                               |    2 +-\n 36 files changed, 72 insertions(+), 72 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex cd17814..52bbafb 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1351,7 +1351,7 @@ receive.denyCurrentBranch::\n \n receive.denyNonFastForwards::\n \tIf set to true, git-receive-pack will deny a ref update which is\n-\tnot a fast forward. Use this to prevent such an update via a push,\n+\tnot a fast-forward. Use this to prevent such an update via a push,\n \teven if that push is forced. This configuration variable is\n \tset when initializing a shared repository.\n \ndiff --git a/Documentation/git-http-push.txt b/Documentation/git-http-push.txt\nindex aef383e..ddf7a18 100644\n--- a/Documentation/git-http-push.txt\n+++ b/Documentation/git-http-push.txt\n@@ -82,11 +82,11 @@ destination side.\n \n Without '--force', the <src> ref is stored at the remote only if\n <dst> does not exist, or <dst> is a proper subset (i.e. an\n-ancestor) of <src>.  This check, known as \"fast forward check\",\n+ancestor) of <src>.  This check, known as \"fast-forward check\",\n is performed in order to avoid accidentally overwriting the\n remote ref and lose other peoples' commits from there.\n \n-With '--force', the fast forward check is disabled for all refs.\n+With '--force', the fast-forward check is disabled for all refs.\n \n Optionally, a <ref> parameter can be prefixed with a plus '+' sign\n to disable the fast-forward check only on that ref.\ndiff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\nindex ba6a8a2..1a9d886 100644\n--- a/Documentation/git-push.txt\n+++ b/Documentation/git-push.txt\n@@ -50,9 +50,9 @@ updated.\n +\n The object referenced by <src> is used to update the <dst> reference\n on the remote side, but by default this is only allowed if the\n-update can fast forward <dst>.  By having the optional leading `{plus}`,\n+update can fast-forward <dst>.  By having the optional leading `{plus}`,\n you can tell git to update the <dst> ref even when the update is not a\n-fast forward.  This does *not* attempt to merge <src> into <dst>.  See\n+fast-forward.  This does *not* attempt to merge <src> into <dst>.  See\n EXAMPLES below for details.\n +\n `tag <tag>` means the same as `refs/tags/<tag>:refs/tags/<tag>`.\n@@ -60,7 +60,7 @@ EXAMPLES below for details.\n Pushing an empty <src> allows you to delete the <dst> ref from\n the remote repository.\n +\n-The special refspec `:` (or `{plus}:` to allow non-fast forward updates)\n+The special refspec `:` (or `{plus}:` to allow non-fast-forward updates)\n directs git to push \"matching\" branches: for every branch that exists on\n the local side, the remote side is updated if a branch of the same name\n already exists on the remote side.  This is the default operation mode\n@@ -171,10 +171,10 @@ summary::\n \tFor a successfully pushed ref, the summary shows the old and new\n \tvalues of the ref in a form suitable for using as an argument to\n \t`git log` (this is `<old>..<new>` in most cases, and\n-\t`<old>...<new>` for forced non-fast forward updates). For a\n+\t`<old>...<new>` for forced non-fast-forward updates). For a\n \tfailed update, more details are given for the failure.\n \tThe string `rejected` indicates that git did not try to send the\n-\tref at all (typically because it is not a fast forward). The\n+\tref at all (typically because it is not a fast-forward). The\n \tstring `remote rejected` indicates that the remote end refused\n \tthe update; this rejection is typically caused by a hook on the\n \tremote side. The string `remote failure` indicates that the\n@@ -342,9 +342,9 @@ git push origin :experimental::\n \n git push origin {plus}dev:master::\n \tUpdate the origin repository's master branch with the dev branch,\n-\tallowing non-fast forward updates.  *This can leave unreferenced\n+\tallowing non-fast-forward updates.  *This can leave unreferenced\n \tcommits dangling in the origin repository.*  Consider the\n-\tfollowing situation, where a fast forward is not possible:\n+\tfollowing situation, where a fast-forward is not possible:\n +\n ----\n \t    o---o---o---A---B  origin/master\ndiff --git a/Documentation/git-read-tree.txt b/Documentation/git-read-tree.txt\nindex 4a932b0..a10ce4b 100644\n--- a/Documentation/git-read-tree.txt\n+++ b/Documentation/git-read-tree.txt\n@@ -144,7 +144,7 @@ Two Tree Merge\n Typically, this is invoked as `git read-tree -m $H $M`, where $H\n is the head commit of the current repository, and $M is the head\n of a foreign tree, which is simply ahead of $H (i.e. we are in a\n-fast forward situation).\n+fast-forward situation).\n \n When two trees are specified, the user is telling 'git-read-tree'\n the following:\ndiff --git a/Documentation/git-receive-pack.txt b/Documentation/git-receive-pack.txt\nindex 514f03c..cb5f405 100644\n--- a/Documentation/git-receive-pack.txt\n+++ b/Documentation/git-receive-pack.txt\n@@ -20,7 +20,7 @@ The UI for the protocol is on the 'git-send-pack' side, and the\n program pair is meant to be used to push updates to remote\n repository.  For pull operations, see linkgit:git-fetch-pack[1].\n \n-The command allows for creation and fast forwarding of sha1 refs\n+The command allows for creation and fast-forwarding of sha1 refs\n (heads/tags) on the remote end (strictly speaking, it is the\n local end 'git-receive-pack' runs, but to the user who is sitting at\n the send-pack end, it is updating the remote.  Confused?)\ndiff --git a/Documentation/git-reset.txt b/Documentation/git-reset.txt\nindex 469cf6d..2d27e40 100644\n--- a/Documentation/git-reset.txt\n+++ b/Documentation/git-reset.txt\n@@ -150,7 +150,7 @@ Automatic merge failed; fix conflicts and then commit the result.\n $ git reset --hard                 <2>\n $ git pull . topic/branch          <3>\n Updating from 41223... to 13134...\n-Fast forward\n+Fast-forward\n $ git reset --hard ORIG_HEAD       <4>\n ------------\n +\n@@ -161,7 +161,7 @@ right now, so you decide to do that later.\n which is a synonym for \"git reset --hard HEAD\" clears the mess\n from the index file and the working tree.\n <3> Merge a topic branch into the current branch, which resulted\n-in a fast forward.\n+in a fast-forward.\n <4> But you decided that the topic branch is not ready for public\n consumption yet.  \"pull\" or \"merge\" always leaves the original\n tip of the current branch in ORIG_HEAD, so resetting hard to it\ndiff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt\nindex 3998218..5a04c6e 100644\n--- a/Documentation/git-send-pack.txt\n+++ b/Documentation/git-send-pack.txt\n@@ -105,11 +105,11 @@ name. See linkgit:git-rev-parse[1].\n \n Without '--force', the <src> ref is stored at the remote only if\n <dst> does not exist, or <dst> is a proper subset (i.e. an\n-ancestor) of <src>.  This check, known as \"fast forward check\",\n+ancestor) of <src>.  This check, known as \"fast-forward check\",\n is performed in order to avoid accidentally overwriting the\n remote ref and lose other peoples' commits from there.\n \n-With '--force', the fast forward check is disabled for all refs.\n+With '--force', the fast-forward check is disabled for all refs.\n \n Optionally, a <ref> parameter can be prefixed with a plus '+' sign\n to disable the fast-forward check only on that ref.\ndiff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\nindex b3640c4..253ef62 100644\n--- a/Documentation/gitcore-tutorial.txt\n+++ b/Documentation/gitcore-tutorial.txt\n@@ -993,7 +993,7 @@ would be different)\n \n ----------------\n Updating from ae3a2da... to a80b4aa....\n-Fast forward (no commit created; -m option ignored)\n+Fast-forward (no commit created; -m option ignored)\n  example |    1 +\n  hello   |    1 +\n  2 files changed, 2 insertions(+), 0 deletions(-)\n@@ -1003,7 +1003,7 @@ Because your branch did not contain anything more than what had\n already been merged into the `master` branch, the merge operation did\n not actually do a merge. Instead, it just updated the top of\n the tree of your branch to that of the `master` branch. This is\n-often called 'fast forward' merge.\n+often called 'fast-forward' merge.\n \n You can run `gitk \\--all` again to see how the commit ancestry\n looks like, or run 'show-branch', which tells you this.\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex 06e0f31..4cc3d13 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -229,7 +229,7 @@ from updating that ref.\n This hook can be used to prevent 'forced' update on certain refs by\n making sure that the object name is a commit object that is a\n descendant of the commit object named by the old object name.\n-That is, to enforce a \"fast forward only\" policy.\n+That is, to enforce a \"fast-forward only\" policy.\n \n It could also be used to log the old..new status.  However, it\n does not know the entire set of branches, so it would end up\ndiff --git a/Documentation/glossary-content.txt b/Documentation/glossary-content.txt\nindex 43d84d1..1f029f8 100644\n--- a/Documentation/glossary-content.txt\n+++ b/Documentation/glossary-content.txt\n@@ -124,7 +124,7 @@ to point at the new commit.\n \tAn evil merge is a <<def_merge,merge>> that introduces changes that\n \tdo not appear in any <<def_parent,parent>>.\n \n-[[def_fast_forward]]fast forward::\n+[[def_fast_forward]]fast-forward::\n \tA fast-forward is a special type of <<def_merge,merge>> where you have a\n \t<<def_revision,revision>> and you are \"merging\" another\n \t<<def_branch,branch>>'s changes that happen to be a descendant of what\n@@ -220,7 +220,7 @@ to point at the new commit.\n \tconflict, manual intervention may be required to complete the\n \tmerge.\n +\n-As a noun: unless it is a <<def_fast_forward,fast forward>>, a\n+As a noun: unless it is a <<def_fast_forward,fast-forward>>, a\n successful merge results in the creation of a new <<def_commit,commit>>\n representing the result of the merge, and having as\n <<def_parent,parents>> the tips of the merged <<def_branch,branches>>.\ndiff --git a/Documentation/howto/maintain-git.txt b/Documentation/howto/maintain-git.txt\nindex 4357e26..d527b30 100644\n--- a/Documentation/howto/maintain-git.txt\n+++ b/Documentation/howto/maintain-git.txt\n@@ -59,7 +59,7 @@ The policy.\n    not yet pass the criteria set for 'next'.\n \n  - The tips of 'master', 'maint' and 'next' branches will always\n-   fast forward, to allow people to build their own\n+   fast-forward, to allow people to build their own\n    customization on top of them.\n \n  - Usually 'master' contains all of 'maint', 'next' contains all\ndiff --git a/Documentation/howto/revert-branch-rebase.txt b/Documentation/howto/revert-branch-rebase.txt\nindex e70d8a3..8c32da6 100644\n--- a/Documentation/howto/revert-branch-rebase.txt\n+++ b/Documentation/howto/revert-branch-rebase.txt\n@@ -85,7 +85,7 @@ Fortunately I did not have to; what I have in the current branch\n \n ------------------------------------------------\n $ git checkout master\n-$ git merge revert-c99 ;# this should be a fast forward\n+$ git merge revert-c99 ;# this should be a fast-forward\n Updating from 10d781b9caa4f71495c7b34963bef137216f86a8 to e3a693c...\n  cache.h        |    8 ++++----\n  commit.c       |    2 +-\n@@ -95,7 +95,7 @@ Updating from 10d781b9caa4f71495c7b34963bef137216f86a8 to e3a693c...\n  5 files changed, 8 insertions(+), 8 deletions(-)\n ------------------------------------------------\n \n-There is no need to redo the test at this point.  We fast forwarded\n+There is no need to redo the test at this point.  We fast-forwarded\n and we know 'master' matches 'revert-c99' exactly.  In fact:\n \n ------------------------------------------------\ndiff --git a/Documentation/howto/update-hook-example.txt b/Documentation/howto/update-hook-example.txt\nindex 697d918..b7f8d41 100644\n--- a/Documentation/howto/update-hook-example.txt\n+++ b/Documentation/howto/update-hook-example.txt\n@@ -76,7 +76,7 @@ case \"$1\" in\n     if expr \"$2\" : '0*$' >/dev/null; then\n       info \"The branch '$1' is new...\"\n     else\n-      # updating -- make sure it is a fast forward\n+      # updating -- make sure it is a fast-forward\n       mb=$(git-merge-base \"$2\" \"$3\")\n       case \"$mb,$2\" in\n         \"$2,$mb\") info \"Update is fast-forward\" ;;\ndiff --git a/Documentation/pull-fetch-param.txt b/Documentation/pull-fetch-param.txt\nindex f9811f2..44d9363 100644\n--- a/Documentation/pull-fetch-param.txt\n+++ b/Documentation/pull-fetch-param.txt\n@@ -11,9 +11,9 @@\n +\n The remote ref that matches <src>\n is fetched, and if <dst> is not empty string, the local\n-ref that matches it is fast forwarded using <src>.\n+ref that matches it is fast-forwarded using <src>.\n If the optional plus `+` is used, the local ref\n-is updated even if it does not result in a fast forward\n+is updated even if it does not result in a fast-forward\n update.\n +\n [NOTE]\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 67ebffa..42d92fd 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1384,7 +1384,7 @@ were merged.\n \n However, if the current branch is a descendant of the other--so every\n commit present in the one is already contained in the other--then git\n-just performs a \"fast forward\"; the head of the current branch is moved\n+just performs a \"fast-forward\"; the head of the current branch is moved\n forward to point at the head of the merged-in branch, without any new\n commits being created.\n \n@@ -1719,7 +1719,7 @@ producing a default commit message documenting the branch and\n repository that you pulled from.\n \n (But note that no such commit will be created in the case of a\n-<<fast-forwards,fast forward>>; instead, your branch will just be\n+<<fast-forwards,fast-forward>>; instead, your branch will just be\n updated to point to the latest commit from the upstream branch.)\n \n The `git pull` command can also be given \".\" as the \"remote\" repository,\n@@ -1943,7 +1943,7 @@ $ git push ssh://yourserver.com/~you/proj.git master\n -------------------------------------------------\n \n As with `git fetch`, `git push` will complain if this does not result in a\n-<<fast-forwards,fast forward>>; see the following section for details on\n+<<fast-forwards,fast-forward>>; see the following section for details on\n handling this case.\n \n Note that the target of a \"push\" is normally a\n@@ -1976,7 +1976,7 @@ details.\n What to do when a push fails\n ~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n \n-If a push would not result in a <<fast-forwards,fast forward>> of the\n+If a push would not result in a <<fast-forwards,fast-forward>> of the\n remote branch, then it will fail with an error like:\n \n -------------------------------------------------\n@@ -2115,7 +2115,7 @@ $ git checkout release && git pull\n \n Important note!  If you have any local changes in these branches, then\n this merge will create a commit object in the history (with no local\n-changes git will simply do a \"Fast forward\" merge).  Many people dislike\n+changes git will simply do a fast-forward merge).  Many people dislike\n the \"noise\" that this creates in the Linux history, so you should avoid\n doing this capriciously in the \"release\" branch, as these noisy commits\n will become part of the permanent history when you ask Linus to pull\n@@ -2729,9 +2729,9 @@ In the previous example, when updating an existing branch, \"git fetch\"\n checks to make sure that the most recent commit on the remote\n branch is a descendant of the most recent commit on your copy of the\n branch before updating your copy of the branch to point at the new\n-commit.  Git calls this process a <<fast-forwards,fast forward>>.\n+commit.  Git calls this process a <<fast-forwards,fast-forward>>.\n \n-A fast forward looks something like this:\n+A fast-forward looks something like this:\n \n ................................................\n  o--o--o--o <-- old head of the branch\ndiff --git a/builtin-fetch--tool.c b/builtin-fetch--tool.c\nindex 3dbdf7a..cd10dbc 100644\n--- a/builtin-fetch--tool.c\n+++ b/builtin-fetch--tool.c\n@@ -97,21 +97,21 @@ static int update_local_ref(const char *name,\n \tstrcpy(newh, find_unique_abbrev(sha1_new, DEFAULT_ABBREV));\n \n \tif (in_merge_bases(current, &updated, 1)) {\n-\t\tfprintf(stderr, \"* %s: fast forward to %s\\n\",\n+\t\tfprintf(stderr, \"* %s: fast-forward to %s\\n\",\n \t\t\tname, note);\n \t\tfprintf(stderr, \"  old..new: %s..%s\\n\", oldh, newh);\n-\t\treturn update_ref_env(\"fast forward\", name, sha1_new, sha1_old);\n+\t\treturn update_ref_env(\"fast-forward\", name, sha1_new, sha1_old);\n \t}\n \tif (!force) {\n \t\tfprintf(stderr,\n-\t\t\t\"* %s: not updating to non-fast forward %s\\n\",\n+\t\t\t\"* %s: not updating to non-fast-forward %s\\n\",\n \t\t\tname, note);\n \t\tfprintf(stderr,\n \t\t\t\"  old...new: %s...%s\\n\", oldh, newh);\n \t\treturn 1;\n \t}\n \tfprintf(stderr,\n-\t\t\"* %s: forcing update to non-fast forward %s\\n\",\n+\t\t\"* %s: forcing update to non-fast-forward %s\\n\",\n \t\tname, note);\n \tfprintf(stderr, \"  old...new: %s...%s\\n\", oldh, newh);\n \treturn update_ref_env(\"forced-update\", name, sha1_new, sha1_old);\ndiff --git a/builtin-fetch.c b/builtin-fetch.c\nindex cb48c57..89cedc8 100644\n--- a/builtin-fetch.c\n+++ b/builtin-fetch.c\n@@ -269,7 +269,7 @@ static int update_local_ref(struct ref *ref,\n \t\tstrcpy(quickref, find_unique_abbrev(current->object.sha1, DEFAULT_ABBREV));\n \t\tstrcat(quickref, \"..\");\n \t\tstrcat(quickref, find_unique_abbrev(ref->new_sha1, DEFAULT_ABBREV));\n-\t\tr = s_update_ref(\"fast forward\", ref, 1);\n+\t\tr = s_update_ref(\"fast-forward\", ref, 1);\n \t\tsprintf(display, \"%c %-*s %-*s -> %s%s\", r ? '!' : ' ',\n \t\t\tSUMMARY_WIDTH, quickref, REFCOL_WIDTH, remote,\n \t\t\tpretty_ref, r ? \"  (unable to update local ref)\" : \"\");\n@@ -287,7 +287,7 @@ static int update_local_ref(struct ref *ref,\n \t\t\tr ? \"unable to update local ref\" : \"forced update\");\n \t\treturn r;\n \t} else {\n-\t\tsprintf(display, \"! %-*s %-*s -> %s  (non fast forward)\",\n+\t\tsprintf(display, \"! %-*s %-*s -> %s  (non fast-forward)\",\n \t\t\tSUMMARY_WIDTH, \"[rejected]\", REFCOL_WIDTH, remote,\n \t\t\tpretty_ref);\n \t\treturn 1;\ndiff --git a/builtin-merge.c b/builtin-merge.c\nindex b6b8428..a595b8b 100644\n--- a/builtin-merge.c\n+++ b/builtin-merge.c\n@@ -166,7 +166,7 @@ static struct option builtin_merge_options[] = {\n \tOPT_BOOLEAN(0, \"commit\", &option_commit,\n \t\t\"perform a commit if the merge succeeds (default)\"),\n \tOPT_BOOLEAN(0, \"ff\", &allow_fast_forward,\n-\t\t\"allow fast forward (default)\"),\n+\t\t\"allow fast-forward (default)\"),\n \tOPT_CALLBACK('s', \"strategy\", &use_strategies, \"strategy\",\n \t\t\"merge strategy to use\", option_parse_strategy),\n \tOPT_CALLBACK('m', \"message\", &merge_msg, \"message\",\n@@ -1013,7 +1013,7 @@ int cmd_merge(int argc, const char **argv, const char *prefix)\n \t\t\t\thex,\n \t\t\t\tfind_unique_abbrev(remoteheads->item->object.sha1,\n \t\t\t\tDEFAULT_ABBREV));\n-\t\tstrbuf_addstr(&msg, \"Fast forward\");\n+\t\tstrbuf_addstr(&msg, \"Fast-forward\");\n \t\tif (have_message)\n \t\t\tstrbuf_addstr(&msg,\n \t\t\t\t\" (no commit created; -m option ignored)\");\n@@ -1031,12 +1031,12 @@ int cmd_merge(int argc, const char **argv, const char *prefix)\n \t} else if (!remoteheads->next && common->next)\n \t\t;\n \t\t/*\n-\t\t * We are not doing octopus and not fast forward.  Need\n+\t\t * We are not doing octopus and not fast-forward.  Need\n \t\t * a real merge.\n \t\t */\n \telse if (!remoteheads->next && !common->next && option_commit) {\n \t\t/*\n-\t\t * We are not doing octopus, not fast forward, and have\n+\t\t * We are not doing octopus, not fast-forward, and have\n \t\t * only one common.\n \t\t */\n \t\trefresh_cache(REFRESH_QUIET);\ndiff --git a/builtin-push.c b/builtin-push.c\nindex 3cb1ee4..e883574 100644\n--- a/builtin-push.c\n+++ b/builtin-push.c\n@@ -159,7 +159,7 @@ static int do_push(const char *repo, int flags)\n \t\terror(\"failed to push some refs to '%s'\", url[i]);\n \t\tif (nonfastforward && advice_push_nonfastforward) {\n \t\t\tprintf(\"To prevent you from losing history, non-fast-forward updates were rejected\\n\"\n-\t\t\t       \"Merge the remote changes before pushing again.  See the 'non-fast forward'\\n\"\n+\t\t\t       \"Merge the remote changes before pushing again.  See the 'non-fast-forward'\\n\"\n \t\t\t       \"section of 'git push --help' for details.\\n\");\n \t\t}\n \t\terrs++;\ndiff --git a/builtin-receive-pack.c b/builtin-receive-pack.c\nindex b771fe9..fea8fcd 100644\n--- a/builtin-receive-pack.c\n+++ b/builtin-receive-pack.c\n@@ -329,9 +329,9 @@ static const char *update(struct command *cmd)\n \t\t\t\tbreak;\n \t\tfree_commit_list(bases);\n \t\tif (!ent) {\n-\t\t\terror(\"denying non-fast forward %s\"\n+\t\t\terror(\"denying non-fast-forward %s\"\n \t\t\t      \" (you should pull first)\", name);\n-\t\t\treturn \"non-fast forward\";\n+\t\t\treturn \"non-fast-forward\";\n \t\t}\n \t}\n \tif (run_update_hook(cmd)) {\ndiff --git a/builtin-remote.c b/builtin-remote.c\nindex 0777dd7..9aafc19 100644\n--- a/builtin-remote.c\n+++ b/builtin-remote.c\n@@ -953,7 +953,7 @@ static int show_push_info_item(struct string_list_item *item, void *cb_data)\n \t\tstatus = \"up to date\";\n \t\tbreak;\n \tcase PUSH_STATUS_FASTFORWARD:\n-\t\tstatus = \"fast forwardable\";\n+\t\tstatus = \"fast-forwardable\";\n \t\tbreak;\n \tcase PUSH_STATUS_OUTOFDATE:\n \t\tstatus = \"local out of date\";\ndiff --git a/builtin-send-pack.c b/builtin-send-pack.c\nindex 37e528e..37acad5 100644\n--- a/builtin-send-pack.c\n+++ b/builtin-send-pack.c\n@@ -246,7 +246,7 @@ static int print_one_push_status(struct ref *ref, const char *dest, int count)\n \t\tbreak;\n \tcase REF_STATUS_REJECT_NONFASTFORWARD:\n \t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n-\t\t\t\t\"non-fast forward\");\n+\t\t\t\t\"non-fast-forward\");\n \t\tbreak;\n \tcase REF_STATUS_REMOTE_REJECT:\n \t\tprint_ref_status('!', \"[remote rejected]\", ref,\ndiff --git a/contrib/examples/git-merge.sh b/contrib/examples/git-merge.sh\nindex e9588ee..500635f 100755\n--- a/contrib/examples/git-merge.sh\n+++ b/contrib/examples/git-merge.sh\n@@ -14,7 +14,7 @@ summary              (synonym to --stat)\n log                  add list of one-line log to merge commit message\n squash               create a single commit instead of doing a merge\n commit               perform a commit if the merge succeeds (default)\n-ff                   allow fast forward (default)\n+ff                   allow fast-forward (default)\n s,strategy=          merge strategy to use\n m,message=           message to be used for the merge commit (if any)\n \"\n@@ -353,7 +353,7 @@ t,1,\"$head\",*)\n \t# Again the most common case of merging one remote.\n \techo \"Updating $(git rev-parse --short $head)..$(git rev-parse --short $1)\"\n \tgit update-index --refresh 2>/dev/null\n-\tmsg=\"Fast forward\"\n+\tmsg=\"Fast-forward\"\n \tif test -n \"$have_message\"\n \tthen\n \t\tmsg=\"$msg (no commit created; -m option ignored)\"\n@@ -365,11 +365,11 @@ t,1,\"$head\",*)\n \texit 0\n \t;;\n ?,1,?*\"$LF\"?*,*)\n-\t# We are not doing octopus and not fast forward.  Need a\n+\t# We are not doing octopus and not fast-forward.  Need a\n \t# real merge.\n \t;;\n ?,1,*,)\n-\t# We are not doing octopus, not fast forward, and have only\n+\t# We are not doing octopus, not fast-forward, and have only\n \t# one common.\n \tgit update-index --refresh 2>/dev/null\n \tcase \"$allow_trivial_merge\" in\ndiff --git a/contrib/examples/git-resolve.sh b/contrib/examples/git-resolve.sh\nindex 0ee1bd8..8f98142 100755\n--- a/contrib/examples/git-resolve.sh\n+++ b/contrib/examples/git-resolve.sh\n@@ -48,7 +48,7 @@ case \"$common\" in\n \"$head\")\n \techo \"Updating $(git rev-parse --short $head)..$(git rev-parse --short $merge)\"\n \tgit read-tree -u -m $head $merge || exit 1\n-\tgit update-ref -m \"resolve $merge_name: Fast forward\" \\\n+\tgit update-ref -m \"resolve $merge_name: Fast-forward\" \\\n \t\tHEAD \"$merge\" \"$head\"\n \tgit diff-tree -p $head $merge | git apply --stat\n \tdropheads\ndiff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\nindex 2a66063..3b075ad 100755\n--- a/contrib/hooks/post-receive-email\n+++ b/contrib/hooks/post-receive-email\n@@ -315,8 +315,8 @@ generate_update_branch_email()\n \t# \"remotes/\" will be ignored as well.\n \n \t# List all of the revisions that were removed by this update, in a\n-\t# fast forward update, this list will be empty, because rev-list O\n-\t# ^N is empty.  For a non fast forward, O ^N is the list of removed\n+\t# fast-forward update, this list will be empty, because rev-list O\n+\t# ^N is empty.  For a non fast-forward, O ^N is the list of removed\n \t# revisions\n \tfast_forward=\"\"\n \trev=\"\"\n@@ -411,7 +411,7 @@ generate_update_branch_email()\n \t# revision because the base is effectively a random revision at this\n \t# point - the user will be interested in what this revision changed\n \t# - including the undoing of previous revisions in the case of\n-\t# non-fast forward updates.\n+\t# non-fast-forward updates.\n \techo \"\"\n \techo \"Summary of changes:\"\n \tgit diff-tree --stat --summary --find-copies-harder $oldrev..$newrev\ndiff --git a/git-gui/lib/branch_create.tcl b/git-gui/lib/branch_create.tcl\nindex 3817771..f1235c7 100644\n--- a/git-gui/lib/branch_create.tcl\n+++ b/git-gui/lib/branch_create.tcl\n@@ -77,7 +77,7 @@ constructor dialog {} {\n \t\t-variable @opt_merge\n \tpack $w.options.merge.no -side left\n \tradiobutton $w.options.merge.ff \\\n-\t\t-text [mc \"Fast Forward Only\"] \\\n+\t\t-text [mc \"Fast-forward Only\"] \\\n \t\t-value ff \\\n \t\t-variable @opt_merge\n \tpack $w.options.merge.ff -side left\ndiff --git a/git-merge-octopus.sh b/git-merge-octopus.sh\nindex 1dadbb4..825c52c 100755\n--- a/git-merge-octopus.sh\n+++ b/git-merge-octopus.sh\n@@ -81,7 +81,7 @@ do\n \t\t# tree as the intermediate result of the merge.\n \t\t# We still need to count this as part of the parent set.\n \n-\t\techo \"Fast forwarding to: $SHA1\"\n+\t\techo \"Fast-forwarding to: $SHA1\"\n \t\tgit read-tree -u -m $head $SHA1 || exit\n \t\tMRC=$SHA1 MRT=$(git write-tree)\n \t\tcontinue\ndiff --git a/git-pull.sh b/git-pull.sh\nindex fc78592..f36eb3e 100755\n--- a/git-pull.sh\n+++ b/git-pull.sh\n@@ -171,7 +171,7 @@ then\n \t# First update the working tree to match $curr_head.\n \n \techo >&2 \"Warning: fetch updated the current branch head.\"\n-\techo >&2 \"Warning: fast forwarding your working tree from\"\n+\techo >&2 \"Warning: fast-forwarding your working tree from\"\n \techo >&2 \"Warning: commit $orig_head.\"\n \tgit update-index -q --refresh\n \tgit read-tree -u -m \"$orig_head\" \"$curr_head\" ||\ndiff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh\nindex 23ded48..c898788 100755\n--- a/git-rebase--interactive.sh\n+++ b/git-rebase--interactive.sh\n@@ -168,7 +168,7 @@ pick_one () {\n \t\toutput git reset --hard $sha1\n \t\ttest \"a$1\" = a-n && output git reset --soft $current_sha1\n \t\tsha1=$(git rev-parse --short $sha1)\n-\t\toutput warn Fast forward to $sha1\n+\t\toutput warn Fast-forward to $sha1\n \telse\n \t\toutput git cherry-pick \"$@\"\n \tfi\n@@ -248,9 +248,9 @@ pick_one_preserving_merges () {\n \tdone\n \tcase $fast_forward in\n \tt)\n-\t\toutput warn \"Fast forward to $sha1\"\n+\t\toutput warn \"Fast-forward to $sha1\"\n \t\toutput git reset --hard $sha1 ||\n-\t\t\tdie \"Cannot fast forward to $sha1\"\n+\t\t\tdie \"Cannot fast-forward to $sha1\"\n \t\t;;\n \tf)\n \t\tfirst_parent=$(expr \"$new_parents\" : ' \\([^ ]*\\)')\ndiff --git a/git-rebase.sh b/git-rebase.sh\nindex 6ec155c..6830e16 100755\n--- a/git-rebase.sh\n+++ b/git-rebase.sh\n@@ -496,7 +496,7 @@ then\n fi\n \n # If the $onto is a proper descendant of the tip of the branch, then\n-# we just fast forwarded.\n+# we just fast-forwarded.\n if test \"$mb\" = \"$branch\"\n then\n \tsay \"Fast-forwarded $branch_name to $onto_name.\"\ndiff --git a/t/t1001-read-tree-m-2way.sh b/t/t1001-read-tree-m-2way.sh\nindex 271bc4e..c2d408b 100755\n--- a/t/t1001-read-tree-m-2way.sh\n+++ b/t/t1001-read-tree-m-2way.sh\n@@ -5,7 +5,7 @@\n \n test_description='Two way merge with read-tree -m $H $M\n \n-This test tries two-way merge (aka fast forward with carry forward).\n+This test tries two-way merge (aka fast-forward with carry forward).\n \n There is the head (called H) and another commit (called M), which is\n simply ahead of H.  The index and the work tree contains a state that\n@@ -51,7 +51,7 @@ check_cache_at () {\n }\n \n cat >bozbar-old <<\\EOF\n-This is a sample file used in two-way fast forward merge\n+This is a sample file used in two-way fast-forward merge\n tests.  Its second line ends with a magic word bozbar\n which will be modified by the merged head to gnusto.\n It has some extra lines so that external tools can\n@@ -300,7 +300,7 @@ test_expect_success \\\n      echo gnusto gnusto >bozbar &&\n      if read_tree_twoway $treeH $treeM; then false; else :; fi'\n \n-# This fails with straight two-way fast forward.\n+# This fails with straight two-way fast-forward.\n test_expect_success \\\n     '22 - local change cache updated.' \\\n     'rm -f .git/index &&\ndiff --git a/t/t5505-remote.sh b/t/t5505-remote.sh\nindex 852ccb5..220b6a3 100755\n--- a/t/t5505-remote.sh\n+++ b/t/t5505-remote.sh\n@@ -158,7 +158,7 @@ cat > test/expect << EOF\n     another\n     master\n   Local refs configured for 'git push':\n-    ahead  forces to master  (fast forwardable)\n+    ahead  forces to master  (fast-forwardable)\n     master pushes to another (up to date)\n EOF\n \ndiff --git a/t/t5518-fetch-exit-status.sh b/t/t5518-fetch-exit-status.sh\nindex c6bc65f..c2060bb 100755\n--- a/t/t5518-fetch-exit-status.sh\n+++ b/t/t5518-fetch-exit-status.sh\n@@ -22,7 +22,7 @@ test_expect_success setup '\n \tgit commit -a -m next\n '\n \n-test_expect_success 'non fast forward fetch' '\n+test_expect_success 'non-fast-forward fetch' '\n \n \ttest_must_fail git fetch . master:side\n \ndiff --git a/t/t6028-merge-up-to-date.sh b/t/t6028-merge-up-to-date.sh\nindex f8f3e3f..a91644e 100755\n--- a/t/t6028-merge-up-to-date.sh\n+++ b/t/t6028-merge-up-to-date.sh\n@@ -1,6 +1,6 @@\n #!/bin/sh\n \n-test_description='merge fast forward and up to date'\n+test_description='merge fast-forward and up to date'\n \n . ./test-lib.sh\n \ndiff --git a/transport.c b/transport.c\nindex 644a30a..d81a42a 100644\n--- a/transport.c\n+++ b/transport.c\n@@ -668,7 +668,7 @@ static int print_one_push_status(struct ref *ref, const char *dest, int count, i\n \t\tbreak;\n \tcase REF_STATUS_REJECT_NONFASTFORWARD:\n \t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n-\t\t\t\t\t\t \"non-fast forward\", porcelain);\n+\t\t\t\t\t\t \"non-fast-forward\", porcelain);\n \t\tbreak;\n \tcase REF_STATUS_REMOTE_REJECT:\n \t\tprint_ref_status('!', \"[remote rejected]\", ref,\ndiff --git a/unpack-trees.c b/unpack-trees.c\nindex 720f7a1..157d5d0 100644\n--- a/unpack-trees.c\n+++ b/unpack-trees.c\n@@ -895,7 +895,7 @@ int threeway_merge(struct cache_entry **stages, struct unpack_trees_options *o)\n  * Two-way merge.\n  *\n  * The rule is to \"carry forward\" what is in the index without losing\n- * information across a \"fast forward\", favoring a successful merge\n+ * information across a \"fast-forward\", favoring a successful merge\n  * over a merge failure when it makes sense.  For details of the\n  * \"carry forward\" rule, please see <Documentation/git-read-tree.txt>.\n  *\n-- \n1.6.5.1\n"},{"id":"125843","messageId":"20091024220709.6117@nanako3.lavabit.com","threadId":"21342","inReplyTo":"1256373092-15126-1-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH] Use fast-forward","fromName":"Nanako Shiraishi","fromEmail":"nanako3@lavabit.com","sentAt":"2009-10-24T13:07:09Z","receivedAt":"2009-10-24T13:07:09Z","isPatch":true,"sender":{"key":"nanako3@lavabit.com","avatar":"https://gravatar.com/avatar/3777b9e201c5883a62b1a6fdf7c53f2d712d1d80989146063ea861e33aad72a8?d=mp&s=160"},"body":"Quoting Felipe Contreras <felipe.contreras@gmail.com>\n\n> As suggested in the mailing list, now I've replaced all instances of 'fast\n> forward' with 'fast-forward'.\n\nI had an impression that the consensus from the previous \ndiscussion was that there is no such consensus that this \nis an improvement, because there isn't a clear-cut rule?\n\n-- \nNanako Shiraishi\nhttp://ivory.ap.teacup.com/nanako3/\n"},{"id":"125846","messageId":"94a0d4530910240717i7c8fe973w2623d095daf04ca6@mail.gmail.com","threadId":"21342","inReplyTo":"20091024220709.6117@nanako3.lavabit.com","subject":"Re: [PATCH] Use fast-forward","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-24T14:17:47Z","receivedAt":"2009-10-24T14:17:47Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sat, Oct 24, 2009 at 4:07 PM, Nanako Shiraishi <nanako3@lavabit.com> wrote:\n> Quoting Felipe Contreras <felipe.contreras@gmail.com>\n>\n>> As suggested in the mailing list, now I've replaced all instances of 'fast\n>> forward' with 'fast-forward'.\n>\n> I had an impression that the consensus from the previous\n> discussion was that there is no such consensus that this\n> is an improvement, because there isn't a clear-cut rule?\n\nI don't think anyone wants to preserve the current inconsistent use,\nand I haven't seen arguments in favor of 'fast forward' either. My\nunderstanding was that the comments were in favor of 'fast-forward'\nbut making it consistent across the code-base:\n\nhttp://marc.info/?l=git&m=125532404330701&w=2\nhttp://marc.info/?l=git&m=125535210203730&w=2\n\n-- \nFelipe Contreras\n"},{"id":"125849","messageId":"7v3a587kc2.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"1256373092-15126-2-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-24T17:50:05Z","receivedAt":"2009-10-24T17:50:05Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Is this meant to replace the previous one that is already queued: a0c0ecb\n(user-manual: use 'fast-forward', 2009-10-11)?\n\nIt seems that these mostly match a mechanical token replacement\n\"s/([fF])ast forward/$1ast-forward/g\" in the Documentation area,\nbut I suspect there may be some manual fixes.\n\nToken-replace is much harder to review than to produce, as the result of\nsuch mechanical substitution needs to be examined to see if each change\nmakes sense individually.\n\nI suspect the patch would have been much easier to the reviewers it it\nstated somewhere in the log message:\n\n (1) how the mechanical change was produced;\n\n (2) what criteria was used to choose between leaving the mechanical\n     change as-is and rewording them manually; and\n\n (3) where these non-mechanical changes are.\n\nHere are the list of paths I looked at (during this sitting which did\nnot go til the end of the patch):\n\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n\nOK\n\n> diff --git a/Documentation/git-http-push.txt b/Documentation/git-http-push.txt\n\nOK\n\n> diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\n\nOK, except for two hunks below I am not absolutely sure.\n\n> @@ -60,7 +60,7 @@ EXAMPLES below for details.\n>  Pushing an empty <src> allows you to delete the <dst> ref from\n>  the remote repository.\n>  +\n> -The special refspec `:` (or `{plus}:` to allow non-fast forward updates)\n> +The special refspec `:` (or `{plus}:` to allow non-fast-forward updates)\n>  directs git to push \"matching\" branches: for every branch that exists on\n>  the local side, the remote side is updated if a branch of the same name\n>  already exists on the remote side.  This is the default operation mode\n\nHmm, is non-fast-forward a yet another compound word?\n\n> @@ -342,9 +342,9 @@ git push origin :experimental::\n\nLikewise.\n\n> diff --git a/Documentation/git-read-tree.txt b/Documentation/git-read-tree.txt\n> diff --git a/Documentation/git-receive-pack.txt b/Documentation/git-receive-pack.txt\n> diff --git a/Documentation/git-reset.txt b/Documentation/git-reset.txt\n> diff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt\n> diff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\n> diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\n> diff --git a/Documentation/glossary-content.txt b/Documentation/glossary-content.txt\n> diff --git a/Documentation/howto/maintain-git.txt b/Documentation/howto/maintain-git.txt\n> diff --git a/Documentation/howto/revert-branch-rebase.txt b/Documentation/howto/revert-branch-rebase.txt\n> diff --git a/Documentation/howto/update-hook-example.txt b/Documentation/howto/update-hook-example.txt\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n\nOK, except for this hunk I am not sure about.\n\n> @@ -2115,7 +2115,7 @@ $ git checkout release && git pull\n>  \n>  Important note!  If you have any local changes in these branches, then\n>  this merge will create a commit object in the history (with no local\n> -changes git will simply do a \"Fast forward\" merge).  Many people dislike\n> +changes git will simply do a fast-forward merge).  Many people dislike\n>  the \"noise\" that this creates in the Linux history, so you should avoid\n>  doing this capriciously in the \"release\" branch, as these noisy commits\n>  will become part of the permanent history when you ask Linus to pull\n\nIt may be Ok not to emphasize this word but that is not about \"fast\nforward\" vs \"fast-forward\".  It is more about \"in this context, this word\ndoes not have to be emphasized\" kind of copy-editing which does not have\nto be limited to the case where the \"word\" is 'fast-forward'.\n\nIt is my policy not to look at one patch for more than 30 minutes in one\nsitting, because I'll get sloppier toward the end, so I'll stop here for\nnow.  Help in reviewing the rest is appreciated.\n"},{"id":"125852","messageId":"94a0d4530910241212s5d644eb6u66c6f96feafcaf10@mail.gmail.com","threadId":"21342","inReplyTo":"7v3a587kc2.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-24T19:12:36Z","receivedAt":"2009-10-24T19:12:36Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sat, Oct 24, 2009 at 8:50 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Is this meant to replace the previous one that is already queued: a0c0ecb\n> (user-manual: use 'fast-forward', 2009-10-11)?\n\nYes.\n\n> It seems that these mostly match a mechanical token replacement\n> \"s/([fF])ast forward/$1ast-forward/g\" in the Documentation area,\n> but I suspect there may be some manual fixes.\n>\n> Token-replace is much harder to review than to produce, as the result of\n> such mechanical substitution needs to be examined to see if each change\n> makes sense individually.\n\nI manually replaced each instance, and reviewed the patch myself. Most\nof the changes are essentially the same, except a few instances:\n\n\"Fast forward\" -> fast-forward\nFast Forward Only -> Fast-forward Only\n\n> I suspect the patch would have been much easier to the reviewers it it\n> stated somewhere in the log message:\n>\n>  (1) how the mechanical change was produced;\n\nThere wasn't such.\n\n>  (2) what criteria was used to choose between leaving the mechanical\n>     change as-is and rewording them manually; and\n\nIf it wasn't straight forward. I considered the following straightforward:\nfast forward -> fast-forward\nfast forwarded -> fast-forwarded\nfast forwarding -> fast-forwarding\nfast forwardable -> fast-forwardable\nnon-fast forward -> non-fast-forward\nFast forward -> Fast-forward\nFast forwarding -> Fast-forwarding\n\n>  (3) where these non-mechanical changes are.\n\nMentioned on the second comment.\n\n> Here are the list of paths I looked at (during this sitting which did\n> not go til the end of the patch):\n>\n>> diff --git a/Documentation/config.txt b/Documentation/config.txt\n>\n> OK\n>\n>> diff --git a/Documentation/git-http-push.txt b/Documentation/git-http-push.txt\n>\n> OK\n>\n>> diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\n>\n> OK, except for two hunks below I am not absolutely sure.\n>\n>> @@ -60,7 +60,7 @@ EXAMPLES below for details.\n>>  Pushing an empty <src> allows you to delete the <dst> ref from\n>>  the remote repository.\n>>  +\n>> -The special refspec `:` (or `{plus}:` to allow non-fast forward updates)\n>> +The special refspec `:` (or `{plus}:` to allow non-fast-forward updates)\n>>  directs git to push \"matching\" branches: for every branch that exists on\n>>  the local side, the remote side is updated if a branch of the same name\n>>  already exists on the remote side.  This is the default operation mode\n>\n> Hmm, is non-fast-forward a yet another compound word?\n\nYes. AFAIK.\n\n>> @@ -342,9 +342,9 @@ git push origin :experimental::\n>\n> Likewise.\n>\n>> diff --git a/Documentation/git-read-tree.txt b/Documentation/git-read-tree.txt\n>> diff --git a/Documentation/git-receive-pack.txt b/Documentation/git-receive-pack.txt\n>> diff --git a/Documentation/git-reset.txt b/Documentation/git-reset.txt\n>> diff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt\n>> diff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\n>> diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\n>> diff --git a/Documentation/glossary-content.txt b/Documentation/glossary-content.txt\n>> diff --git a/Documentation/howto/maintain-git.txt b/Documentation/howto/maintain-git.txt\n>> diff --git a/Documentation/howto/revert-branch-rebase.txt b/Documentation/howto/revert-branch-rebase.txt\n>> diff --git a/Documentation/howto/update-hook-example.txt b/Documentation/howto/update-hook-example.txt\n>> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n>\n> OK, except for this hunk I am not sure about.\n>\n>> @@ -2115,7 +2115,7 @@ $ git checkout release && git pull\n>>\n>>  Important note!  If you have any local changes in these branches, then\n>>  this merge will create a commit object in the history (with no local\n>> -changes git will simply do a \"Fast forward\" merge).  Many people dislike\n>> +changes git will simply do a fast-forward merge).  Many people dislike\n>>  the \"noise\" that this creates in the Linux history, so you should avoid\n>>  doing this capriciously in the \"release\" branch, as these noisy commits\n>>  will become part of the permanent history when you ask Linus to pull\n>\n> It may be Ok not to emphasize this word but that is not about \"fast\n> forward\" vs \"fast-forward\".  It is more about \"in this context, this word\n> does not have to be emphasized\" kind of copy-editing which does not have\n> to be limited to the case where the \"word\" is 'fast-forward'.\n\nI couldn't parse that. From what I can see \"Fast forward\" was\nemphasized because the author thought the words didn't make much sense\nseparated. Now that the word is fast-forward, there's no need to\nemphasize.\n\nCheers.\n\n-- \nFelipe Contreras\n"},{"id":"125856","messageId":"7v3a5860gr.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"94a0d4530910241212s5d644eb6u66c6f96feafcaf10@mail.gmail.com","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-24T19:44:36Z","receivedAt":"2009-10-24T19:44:36Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n>> I suspect the patch would have been much easier to the reviewers it it\n>> stated somewhere in the log message:\n>>\n>>  (1) how the mechanical change was produced;\n>\n> There wasn't such.\n\nThat is actually a bad news; it is even worse than mechanical spotting\nfollowed by manual inspection.  It would force us feel _more_ worried, as\nwe would then need to grep for leftovers that your manual conversion may\nnot have caught.  Sigh...\n\n>>  (2) what criteria was used to choose between leaving the mechanical\n>>     change as-is and rewording them manually; and\n>\n> If it wasn't straight forward. I considered the following straightforward:\n> fast forward -> fast-forward\n> fast forwarded -> fast-forwarded\n> fast forwarding -> fast-forwarding\n> fast forwardable -> fast-forwardable\n> non-fast forward -> non-fast-forward\n> Fast forward -> Fast-forward\n> Fast forwarding -> Fast-forwarding\n\nAll of these are what \"s/([fF])ast forward/$1ast-forward/g\" does, aren't\nthey?\n\n> I couldn't parse that. From what I can see \"Fast forward\" was\n> emphasized because the author thought the words didn't make much sense\n> separated. Now that the word is fast-forward, there's no need to\n> emphasize.\n\nEven after your patch, hunk beginning on line 1384 of the\nuser-manual says\n\n    ... then git just performs a \"fast-forward\"; the head of the ...\n\nand I think you did the right thing by keeping these dq here.  This is the\nfirst occurrence of the word followed by its explanation and for that\nreason, the word deserves to be emphasized---IOW, the context calls for an\nemphasis.\n\nIn the \"Important note!\" part, we talk about the pull operation that\nnormally creates a merge commit, and _in contrast_, under a particular\ncondition (namely, \"no local changes\"), it does something different\n(namely, a \"fast-forward\").  We should keep the emphasis on \"fast-forward\"\nhere for exactly the same reason---the context calls for an emphasis.\n\nI was about to read the rest of the patch (the non-doc part) but instead I\nended up spending another 20 minutes writing this message, so the review\nby me on the remainder has to wait til sometime later.\n"},{"id":"125868","messageId":"7v1vks2l8q.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"1256373092-15126-2-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-25T03:41:41Z","receivedAt":"2009-10-25T03:41:41Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"This is the latter half of my review; please see\n\n    From:\tJunio C Hamano <gitster@pobox.com>\n    Subject: Re: [PATCH] Use 'fast-forward' all over the place\n    Date:\tSat, 24 Oct 2009 10:50:05 -0700\n    Message-ID: <7v3a587kc2.fsf@alter.siamese.dyndns.org>\n\nfor the other half.\n\nFelipe Contreras <felipe.contreras@gmail.com> writes:\n\n> diff --git a/builtin-fetch--tool.c b/builtin-fetch--tool.c\n\nOk.  We may want to consider moving this command to contrib/examples/ or\nremoving it, but that is a separate issue and can be done after we apply\nthis patch.\n\n> diff --git a/builtin-fetch.c b/builtin-fetch.c\n> @@ -269,7 +269,7 @@ static int update_local_ref(struct ref *ref,\n>  \t\tstrcpy(quickref, find_unique_abbrev(current->object.sha1, DEFAULT_ABBREV));\n>  \t\tstrcat(quickref, \"..\");\n>  \t\tstrcat(quickref, find_unique_abbrev(ref->new_sha1, DEFAULT_ABBREV));\n> -\t\tr = s_update_ref(\"fast forward\", ref, 1);\n> +\t\tr = s_update_ref(\"fast-forward\", ref, 1);\n\nThis is creating a message in the reflog; I do not think we have any\nscript/program in-tree that relies on the exact wording of this one, so we\nare probably safe ourselves, but I do not know about third-party scripts.\n\nI'd be surprised if there are somebody who reads the reflog and acts\ndifferently upon seeing that we fast-forwarded, but we'd never know until\nwe cook this in 'next' and see people complain.\n\nThe other hunk to this file is an end-user message from Porcelain, so it\nshould be Ok.\n\n> diff --git a/builtin-merge.c b/builtin-merge.c\n> diff --git a/builtin-push.c b/builtin-push.c\n> @@ -159,7 +159,7 @@ static int do_push(const char *repo, int flags)\n>  \t\terror(\"failed to push some refs to '%s'\", url[i]);\n>  \t\tif (nonfastforward && advice_push_nonfastforward) {\n>  \t\t\tprintf(\"To prevent you from losing history, non-fast-forward updates were rejected\\n\"\n> -\t\t\t       \"Merge the remote changes before pushing again.  See the 'non-fast forward'\\n\"\n> +\t\t\t       \"Merge the remote changes before pushing again.  See the 'non-fast-forward'\\n\"\n>  \t\t\t       \"section of 'git push --help' for details.\\n\");\n>  \t\t}\n\nOk, except that I think we will be seeing merge conflicts with changes\nthat are pending for 1.7.0 with this hunk---I am not looking forward to\nit, but I'll survive with help from rerere.\n\n> diff --git a/builtin-receive-pack.c b/builtin-receive-pack.c\n> diff --git a/builtin-remote.c b/builtin-remote.c\n> diff --git a/builtin-send-pack.c b/builtin-send-pack.c\n> diff --git a/contrib/examples/git-merge.sh b/contrib/examples/git-merge.sh\n> diff --git a/contrib/examples/git-resolve.sh b/contrib/examples/git-resolve.sh\n\nOk.\n\n> diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\n> @@ -315,8 +315,8 @@ generate_update_branch_email()\n>  \t# \"remotes/\" will be ignored as well.\n>  \n>  \t# List all of the revisions that were removed by this update, in a\n> -\t# fast forward update, this list will be empty, because rev-list O\n> -\t# ^N is empty.  For a non fast forward, O ^N is the list of removed\n> +\t# fast-forward update, this list will be empty, because rev-list O\n> +\t# ^N is empty.  For a non fast-forward, O ^N is the list of removed\n\nWasn't non-fast-forward a single compound word of three?\n\n> @@ -411,7 +411,7 @@ generate_update_branch_email()\n>  \t# revision because the base is effectively a random revision at this\n>  \t# point - the user will be interested in what this revision changed\n>  \t# - including the undoing of previous revisions in the case of\n> -\t# non-fast forward updates.\n> +\t# non-fast-forward updates.\n>  \techo \"\"\n>  \techo \"Summary of changes:\"\n>  \tgit diff-tree --stat --summary --find-copies-harder $oldrev..$newrev\n\n... like this hunk, that is.\n\n> diff --git a/git-gui/lib/branch_create.tcl b/git-gui/lib/branch_create.tcl\n> index 3817771..f1235c7 100644\n> --- a/git-gui/lib/branch_create.tcl\n> +++ b/git-gui/lib/branch_create.tcl\n> @@ -77,7 +77,7 @@ constructor dialog {} {\n>  \t\t-variable @opt_merge\n>  \tpack $w.options.merge.no -side left\n>  \tradiobutton $w.options.merge.ff \\\n> -\t\t-text [mc \"Fast Forward Only\"] \\\n> +\t\t-text [mc \"Fast-forward Only\"] \\\n>  \t\t-value ff \\\n>  \t\t-variable @opt_merge\n>  \tpack $w.options.merge.ff -side left\n\nPlease leave git-gui out; (1) it is not managed by me and the patch should\nbe fed to Shawn separately, and (2) updating [mc] keystrings must need\nmatching updates to the translation file and the templates.\n\nI also think this is a label string and the capitalization must be kept,\ni.e. \"Fast-Forward Only\".\n\n> diff --git a/git-merge-octopus.sh b/git-merge-octopus.sh\n> diff --git a/git-pull.sh b/git-pull.sh\n> diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh\n> diff --git a/git-rebase.sh b/git-rebase.sh\n\nOk.\n\n> diff --git a/t/t1001-read-tree-m-2way.sh b/t/t1001-read-tree-m-2way.sh\n> @@ -51,7 +51,7 @@ check_cache_at () {\n>  }\n>  \n>  cat >bozbar-old <<\\EOF\n> -This is a sample file used in two-way fast forward merge\n> +This is a sample file used in two-way fast-forward merge\n>  tests.  Its second line ends with a magic word bozbar\n>  which will be modified by the merged head to gnusto.\n>  It has some extra lines so that external tools can\n\nDoesn't changing this change the actual test blob used?  Do you know if\nthe test still passes when git is not broken?\n\nThe rest of the patches to t/ directory looked Ok.\n\n> diff --git a/transport.c b/transport.c\n> diff --git a/unpack-trees.c b/unpack-trees.c\n\nOk.\n\nThanks.\n"},{"id":"125869","messageId":"7vtyxoymfa.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"7v1vks2l8q.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-25T07:15:05Z","receivedAt":"2009-10-25T07:15:05Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"I'll queue your patch with the following interdiff for now.\n\n * user-manual.txt: do not change emphasis (typography) in this patch---if\n   it needs to be changed, that should be a separate patch.\n\n * builtin-fetch.c: an instance of \"non fast-forward\" was left\n   unconverted; fix it.\n\n * contrib/hooks/post-receive-email: likewise.\n\n   By the way, in Documentation/git-init.txt there also is another \"non\n   fast-forward\"; since it is not a fix-up to this patch, I left it as-is.\n   We may want to change it as well while we are at it.\n\n * git-gui: this is a straight revert of your patch, so that Shawn can\n   apply an undated version of this patch to his tree first, and then I\n   can pull the result from him.\n\n   $ git grep -i -n -e 'fast forward' -- git-gui/\n\n   will show what need to be changed.  This is a key string of \"mc\", so\n   msgid in *.po files and git-gui.pot file need to consistently change,\n   even though the translated strings in *.po files may stay as before.\n\nIf you spot misconversions in here (or other \"fast forward\" that need to be\nconverted but I missed), please holler.\n\nThanks.\n\n---\n\n Documentation/user-manual.txt    |    2 +-\n builtin-fetch.c                  |    2 +-\n contrib/hooks/post-receive-email |    2 +-\n git-gui/lib/branch_create.tcl    |    2 +-\n 4 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 42d92fd..269ec47 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -2115,7 +2115,7 @@ $ git checkout release && git pull\n \n Important note!  If you have any local changes in these branches, then\n this merge will create a commit object in the history (with no local\n-changes git will simply do a fast-forward merge).  Many people dislike\n+changes git will simply do a \"fast-forward\" merge).  Many people dislike\n the \"noise\" that this creates in the Linux history, so you should avoid\n doing this capriciously in the \"release\" branch, as these noisy commits\n will become part of the permanent history when you ask Linus to pull\ndiff --git a/builtin-fetch.c b/builtin-fetch.c\nindex 89cedc8..6303aa0 100644\n--- a/builtin-fetch.c\n+++ b/builtin-fetch.c\n@@ -287,7 +287,7 @@ static int update_local_ref(struct ref *ref,\n \t\t\tr ? \"unable to update local ref\" : \"forced update\");\n \t\treturn r;\n \t} else {\n-\t\tsprintf(display, \"! %-*s %-*s -> %s  (non fast-forward)\",\n+\t\tsprintf(display, \"! %-*s %-*s -> %s  (non-fast-forward)\",\n \t\t\tSUMMARY_WIDTH, \"[rejected]\", REFCOL_WIDTH, remote,\n \t\t\tpretty_ref);\n \t\treturn 1;\ndiff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\nindex 3b075ad..58a35c8 100755\n--- a/contrib/hooks/post-receive-email\n+++ b/contrib/hooks/post-receive-email\n@@ -316,7 +316,7 @@ generate_update_branch_email()\n \n \t# List all of the revisions that were removed by this update, in a\n \t# fast-forward update, this list will be empty, because rev-list O\n-\t# ^N is empty.  For a non fast-forward, O ^N is the list of removed\n+\t# ^N is empty.  For a non-fast-forward, O ^N is the list of removed\n \t# revisions\n \tfast_forward=\"\"\n \trev=\"\"\ndiff --git a/git-gui/lib/branch_create.tcl b/git-gui/lib/branch_create.tcl\nindex f1235c7..3817771 100644\n--- a/git-gui/lib/branch_create.tcl\n+++ b/git-gui/lib/branch_create.tcl\n@@ -77,7 +77,7 @@ constructor dialog {} {\n \t\t-variable @opt_merge\n \tpack $w.options.merge.no -side left\n \tradiobutton $w.options.merge.ff \\\n-\t\t-text [mc \"Fast-forward Only\"] \\\n+\t\t-text [mc \"Fast Forward Only\"] \\\n \t\t-value ff \\\n \t\t-variable @opt_merge\n \tpack $w.options.merge.ff -side left\n"},{"id":"125871","messageId":"7v4opoym7w.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"7vtyxoymfa.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-25T07:19:31Z","receivedAt":"2009-10-25T07:19:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> I'll queue your patch with the following interdiff for now.\n\nSorry, I think I sent a wrong interdiff.\n"},{"id":"125872","messageId":"7vzl7gx7hq.fsf@alter.siamese.dyndns.org","threadId":"21342","inReplyTo":"7v4opoym7w.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-25T07:22:57Z","receivedAt":"2009-10-25T07:22:57Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> I'll queue your patch with the following interdiff for now.\n>\n> Sorry, I think I sent a wrong interdiff.\n\nNevermind; the one I sent out was the right one after all.\n"},{"id":"125873","messageId":"94a0d4530910250156h1dfe345by51581b2e8acd6629@mail.gmail.com","threadId":"21342","inReplyTo":"7v3a5860gr.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-25T08:56:37Z","receivedAt":"2009-10-25T08:56:37Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sat, Oct 24, 2009 at 10:44 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>>> I suspect the patch would have been much easier to the reviewers it it\n>>> stated somewhere in the log message:\n>>>\n>>>  (1) how the mechanical change was produced;\n>>\n>> There wasn't such.\n>\n> That is actually a bad news; it is even worse than mechanical spotting\n> followed by manual inspection.  It would force us feel _more_ worried, as\n> we would then need to grep for leftovers that your manual conversion may\n> not have caught.  Sigh...\n\nPerhaps next time I'll do one round manual, and another one automatic,\nbut in this case I didn't think it was so difficult to review hunk by\nhunk.\n\n>>>  (2) what criteria was used to choose between leaving the mechanical\n>>>     change as-is and rewording them manually; and\n>>\n>> If it wasn't straight forward. I considered the following straightforward:\n>> fast forward -> fast-forward\n>> fast forwarded -> fast-forwarded\n>> fast forwarding -> fast-forwarding\n>> fast forwardable -> fast-forwardable\n>> non-fast forward -> non-fast-forward\n>> Fast forward -> Fast-forward\n>> Fast forwarding -> Fast-forwarding\n>\n> All of these are what \"s/([fF])ast forward/$1ast-forward/g\" does, aren't\n> they?\n\nI guess, yes. But that's not how I did it, so I couldn't be sure.\n\n>> I couldn't parse that. From what I can see \"Fast forward\" was\n>> emphasized because the author thought the words didn't make much sense\n>> separated. Now that the word is fast-forward, there's no need to\n>> emphasize.\n>\n> Even after your patch, hunk beginning on line 1384 of the\n> user-manual says\n>\n>    ... then git just performs a \"fast-forward\"; the head of the ...\n>\n> and I think you did the right thing by keeping these dq here.  This is the\n> first occurrence of the word followed by its explanation and for that\n> reason, the word deserves to be emphasized---IOW, the context calls for an\n> emphasis.\n\nYes, exactly.\n\n> In the \"Important note!\" part, we talk about the pull operation that\n> normally creates a merge commit, and _in contrast_, under a particular\n> condition (namely, \"no local changes\"), it does something different\n> (namely, a \"fast-forward\").  We should keep the emphasis on \"fast-forward\"\n> here for exactly the same reason---the context calls for an emphasis\n\nI don't think so. The emphasis in this case breaks the readability of\nthe text for no reason:\nwith no local changes git will simply do a fast-forward merge\n\nCan be perfectly understood as it is. But in any case, that's out of\nthe scope of this patch.\n\nCheers.\n\n-- \nFelipe Contreras\n"},{"id":"125874","messageId":"94a0d4530910250202h7a890575md080a554f9e0b3e6@mail.gmail.com","threadId":"21342","inReplyTo":"7v1vks2l8q.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-25T09:02:27Z","receivedAt":"2009-10-25T09:02:27Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sun, Oct 25, 2009 at 6:41 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> This is the latter half of my review; please see\n>\n>    From:       Junio C Hamano <gitster@pobox.com>\n>    Subject: Re: [PATCH] Use 'fast-forward' all over the place\n>    Date:       Sat, 24 Oct 2009 10:50:05 -0700\n>    Message-ID: <7v3a587kc2.fsf@alter.siamese.dyndns.org>\n>\n> for the other half.\n>\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n>> diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\n>> @@ -315,8 +315,8 @@ generate_update_branch_email()\n>>       # \"remotes/\" will be ignored as well.\n>>\n>>       # List all of the revisions that were removed by this update, in a\n>> -     # fast forward update, this list will be empty, because rev-list O\n>> -     # ^N is empty.  For a non fast forward, O ^N is the list of removed\n>> +     # fast-forward update, this list will be empty, because rev-list O\n>> +     # ^N is empty.  For a non fast-forward, O ^N is the list of removed\n>\n> Wasn't non-fast-forward a single compound word of three?\n\nYes, but as I mentioned before; it's not something I'm tackling on\nthis patch: only \"non-fast forward\". I think a separate patch should\nbe done for \"non fast.forward\" -> non-fast-forward.\n\n>> diff --git a/git-gui/lib/branch_create.tcl b/git-gui/lib/branch_create.tcl\n>> index 3817771..f1235c7 100644\n>> --- a/git-gui/lib/branch_create.tcl\n>> +++ b/git-gui/lib/branch_create.tcl\n>> @@ -77,7 +77,7 @@ constructor dialog {} {\n>>               -variable @opt_merge\n>>       pack $w.options.merge.no -side left\n>>       radiobutton $w.options.merge.ff \\\n>> -             -text [mc \"Fast Forward Only\"] \\\n>> +             -text [mc \"Fast-forward Only\"] \\\n>>               -value ff \\\n>>               -variable @opt_merge\n>>       pack $w.options.merge.ff -side left\n>\n> Please leave git-gui out; (1) it is not managed by me and the patch should\n> be fed to Shawn separately, and (2) updating [mc] keystrings must need\n> matching updates to the translation file and the templates.\n>\n> I also think this is a label string and the capitalization must be kept,\n> i.e. \"Fast-Forward Only\".\n\nOk.\n\n>> diff --git a/t/t1001-read-tree-m-2way.sh b/t/t1001-read-tree-m-2way.sh\n>> @@ -51,7 +51,7 @@ check_cache_at () {\n>>  }\n>>\n>>  cat >bozbar-old <<\\EOF\n>> -This is a sample file used in two-way fast forward merge\n>> +This is a sample file used in two-way fast-forward merge\n>>  tests.  Its second line ends with a magic word bozbar\n>>  which will be modified by the merged head to gnusto.\n>>  It has some extra lines so that external tools can\n>\n> Doesn't changing this change the actual test blob used?  Do you know if\n> the test still passes when git is not broken?\n\nI didn't check. I will.\n\nCheers.\n\n-- \nFelipe Contreras\n"},{"id":"125875","messageId":"94a0d4530910250206v4827941cs12b88ec75975224e@mail.gmail.com","threadId":"21342","inReplyTo":"7vtyxoymfa.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Use 'fast-forward' all over the place","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-25T09:06:16Z","receivedAt":"2009-10-25T09:06:16Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sun, Oct 25, 2009 at 10:15 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> I'll queue your patch with the following interdiff for now.\n>\n>  * user-manual.txt: do not change emphasis (typography) in this patch---if\n>   it needs to be changed, that should be a separate patch.\n>\n>  * builtin-fetch.c: an instance of \"non fast-forward\" was left\n>   unconverted; fix it.\n>\n>  * contrib/hooks/post-receive-email: likewise.\n>\n>   By the way, in Documentation/git-init.txt there also is another \"non\n>   fast-forward\"; since it is not a fix-up to this patch, I left it as-is.\n>   We may want to change it as well while we are at it.\n>\n>  * git-gui: this is a straight revert of your patch, so that Shawn can\n>   apply an undated version of this patch to his tree first, and then I\n>   can pull the result from him.\n>\n>   $ git grep -i -n -e 'fast forward' -- git-gui/\n>\n>   will show what need to be changed.  This is a key string of \"mc\", so\n>   msgid in *.po files and git-gui.pot file need to consistently change,\n>   even though the translated strings in *.po files may stay as before.\n>\n> If you spot misconversions in here (or other \"fast forward\" that need to be\n> converted but I missed), please holler.\n\nIt's fine by me :)\n\n-- \nFelipe Contreras\n"}]}