{"thread":{"id":"34783","subject":"[PATCH v2 0/11] Modernize user-manual","startedAt":"2013-08-27T17:52:54Z","lastAt":"2013-08-29T18:16:42Z","messageCount":28,"participants":["Thomas Ackermann","Junio C Hamano","Jonathan Nieder","Lars Gullik Bjønnes"],"isPatch":true,"patchVersion":2,"patchTotal":11},"messages":[{"id":"226023","messageId":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":null,"subject":"[PATCH v2 0/11] Modernize user-manual","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:52:54Z","receivedAt":"2013-08-27T17:52:54Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"Hi,\n\nthis is v2 of my patches for user-manual.txt.\n\nThanks to Junio, Jonathan, Martin and Philip for your comments!\n\nI think Philips remarks should be part of a separate patch series which then\nalso should address the current differences in layout between user-manual.html\nand user-manual.pdf.\n\nChanges in v2:\n\n[PATCH 01/13] Call it \"Git User Manual\" and remove reference to very old Git version \n-> same as v1\n[PATCH 02/13] Use current \"detached HEAD\" message \n-> fixed another occurence of \"(no branch)\"\n[PATCH 03/13] Use current output for \"git repack\" \n-> add location where packfile is created\n[PATCH 04/13] Use \"git merge\" instead of \"git pull .\" \n-> reverted changes in \"Getting updates with git pull\"\n[PATCH 05/13] Fix some typos and improve wording\n-> better wording\n[PATCH 06/13] Simplify \"How to make a commit\" \n-> better wording\n[PATCH 07/13] Improve description in \"How to merge\" \n-> this patch was dropped\n[PATCH 08/13] Improve section \"Manipulating branches\" \n-> add missing case for \"current\"\n[PATCH 09/13] Improve section \"Merge multiple trees\" \n-> better wording\n[PATCH 10/13] Remove unnecessary historical note from \"Object storage format\"\n-> same as in v1\n[PATCH 11/13] Remove obscure reference from \"Examples\"\n-> this patch was dropped\n[PATCH 12/13] Remove irrelevant reference from \"Tying it all together\"\n-> same as in v1\n[PATCH 13/13] \"git prune\" is safe now\n-> weakend warning for \"git prune\"\n\nSigned-off-by: Thomas Ackermann \n\n---\nThomas\n"},{"id":"226024","messageId":"1972849803.34382.1377626115372.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 01/11] Call it \"Git User Manual\" and remove reference to very old Git version","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:55:15Z","receivedAt":"2013-08-27T17:55:15Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nReviewed-by: Jonathan Nieder \nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 5 ++---\n 1 file changed, 2 insertions(+), 3 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex fe723e4..103ec9a 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1,6 +1,5 @@\n-Git User's Manual (for version 1.5.3 or newer)\n-______________________________________________\n-\n+Git User Manual\n+_______________\n \n Git is a fast distributed revision control system.\n \n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226025","messageId":"1022906213.34401.1377626164272.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 02/11] Use current \"detached HEAD\" message","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:56:04Z","receivedAt":"2013-08-27T17:56:04Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 19 +++++++++++++------\n 1 file changed, 13 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 103ec9a..bdefd9a 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -312,10 +312,17 @@ referenced by a tag:\n \n ------------------------------------------------\n $ git checkout v2.6.17\n-Note: moving to \"v2.6.17\" which isn't a local branch\n-If you want to create a new branch from this checkout, you may do so\n-(now or later) by using -b with the checkout command again. Example:\n- git checkout -b \n+Note: checking out 'v2.6.17'.\n+\n+You are in 'detached HEAD' state. You can look around, make experimental\n+changes and commit them, and you can discard any commits you make in this\n+state without impacting any branches by performing another checkout.\n+\n+If you want to create a new branch to retain commits you create, you may\n+do so (now or later) by using -b with the checkout command again. Example:\n+\n+ git checkout -b new_branch_name\n+\n HEAD is now at 427abfa... Linux v2.6.17\n ------------------------------------------------\n \n@@ -326,7 +333,7 @@ and git branch shows that you are no longer on a branch:\n $ cat .git/HEAD\n 427abfa28afedffadfca9dd8b067eb6d36bac53f\n $ git branch\n-* (no branch)\n+* (detached from v2.6.17)\n master\n ------------------------------------------------\n \n@@ -3639,7 +3646,7 @@ working on a branch.\n \n -------------------------------------------------\n $ git branch\n-* (no branch)\n+* (detached from d266b98)\n master\n -------------------------------------------------\n \n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226026","messageId":"1431109393.34423.1377626210746.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 03/11] Use current output for \"git repack\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:56:50Z","receivedAt":"2013-08-27T17:56:50Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 16 +++++++---------\n 1 file changed, 7 insertions(+), 9 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex bdefd9a..3f44ca0 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -3203,17 +3203,15 @@ To put the loose objects into a pack, just run git repack:\n \n ------------------------------------------------\n $ git repack\n-Generating pack...\n-Done counting 6020 objects.\n-Deltifying 6020 objects.\n- 100% (6020/6020) done\n-Writing 6020 objects.\n- 100% (6020/6020) done\n-Total 6020, written 6020 (delta 4070), reused 0 (delta 0)\n-Pack pack-3e54ad29d5b2e05838c75df582c65257b8d08e1c created.\n+Counting objects: 6020, done.\n+Delta compression using up to 4 threads.\n+Compressing objects: 100% (6020/6020), done.\n+Writing objects: 100% (6020/6020), done.\n+Total 6020 (delta 4070), reused 0 (delta 0)\n ------------------------------------------------\n \n-You can then run\n+This creates a single \"pack file\" in .git/objects/pack/ \n+containing all currently unpacked objects. You can then run\n \n ------------------------------------------------\n $ git prune\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226027","messageId":"1685937666.34465.1377626298173.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 04/11] Use \"git merge\" instead of \"git pull .\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:58:18Z","receivedAt":"2013-08-27T17:58:18Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\n\"git pull .\" works, but \"git merge\" is the recommended\nway for new users to do things. (The old description\nalso should have read \"The former is actually *not* very\ncommonly used\".)\n\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 3f44ca0..6241a43 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1793,7 +1793,7 @@ $ git pull . branch\n $ git merge branch\n -------------------------------------------------\n \n-are roughly equivalent. The former is actually very commonly used.\n+are roughly equivalent.\n \n [[submitting-patches]]\n Submitting patches to a project\n@@ -2255,11 +2255,11 @@ commit to this branch.\n $ ... patch ... test ... commit [ ... patch ... test ... commit ]*\n -------------------------------------------------\n \n-When you are happy with the state of this change, you can pull it into the\n+When you are happy with the state of this change, you can merge it into the\n \"test\" branch in preparation to make it public:\n \n -------------------------------------------------\n-$ git checkout test && git pull . speed-up-spinlocks\n+$ git checkout test && git merge speed-up-spinlocks\n -------------------------------------------------\n \n It is unlikely that you would have any conflicts here ... but you might if you\n@@ -2271,7 +2271,7 @@ see the value of keeping each patch (or patch series) in its own branch. It\n means that the patches can be moved into the `release` tree in any order.\n \n -------------------------------------------------\n-$ git checkout release && git pull . speed-up-spinlocks\n+$ git checkout release && git merge speed-up-spinlocks\n -------------------------------------------------\n \n After a while, you will have a number of branches, and despite the\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226028","messageId":"70864071.34489.1377626361141.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 05/11] Fix some typos and improve wording","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T17:59:21Z","receivedAt":"2013-08-27T17:59:21Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 20 ++++++++++----------\n 1 file changed, 10 insertions(+), 10 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 6241a43..465d9cb 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -219,7 +219,7 @@ of development leading to that point.\n \n The best way to see how this works is using the linkgit:gitk[1]\n command; running gitk now on a Git repository and looking for merge\n-commits will help understand how the Git organizes history.\n+commits will help understand how Git organizes history.\n \n In the following, we say that commit X is \"reachable\" from commit Y\n if commit X is an ancestor of commit Y. Equivalently, you could say\n@@ -793,7 +793,7 @@ e05db0fd4f31dde7005f075a84f96b360d05984b\n -------------------------------------------------\n \n Or you could recall that the `...` operator selects all commits\n-contained reachable from either one reference or the other but not\n+reachable from either one reference or the other but not\n both; so\n \n -------------------------------------------------\n@@ -820,7 +820,7 @@ You could just visually inspect the commits since e05db0fd:\n $ gitk e05db0fd..\n -------------------------------------------------\n \n-Or you can use linkgit:git-name-rev[1], which will give the commit a\n+or you can use linkgit:git-name-rev[1], which will give the commit a\n name based on any tag it finds pointing to one of the commit's\n descendants:\n \n@@ -864,8 +864,8 @@ because it outputs only commits that are not reachable from v1.5.0-rc1.\n \n As yet another alternative, the linkgit:git-show-branch[1] command lists\n the commits reachable from its arguments with a display on the left-hand\n-side that indicates which arguments that commit is reachable from. So,\n-you can run something like\n+side that indicates which arguments that commit is reachable from. \n+So, if you run something like\n \n -------------------------------------------------\n $ git show-branch e05db0fd v1.5.0-rc0 v1.5.0-rc1 v1.5.0-rc2\n@@ -877,15 +877,15 @@ available\n ...\n -------------------------------------------------\n \n-then search for a line that looks like\n+then a line like\n \n -------------------------------------------------\n + ++ [e05db0fd] Fix warnings in sha1_file.c - use C99 printf format if\n available\n -------------------------------------------------\n \n-Which shows that e05db0fd is reachable from itself, from v1.5.0-rc1, and\n-from v1.5.0-rc2, but not from v1.5.0-rc0.\n+shows that e05db0fd is reachable from itself, from v1.5.0-rc1,\n+and from v1.5.0-rc2, and not from v1.5.0-rc0.\n \n [[showing-commits-unique-to-a-branch]]\n Showing commits unique to a given branch\n@@ -3542,7 +3542,7 @@ with Git 1.5.2 can look up the submodule commits in the repository and\n manually check them out; earlier versions won't recognize the submodules at\n all.\n \n-To see how submodule support works, create (for example) four example\n+To see how submodule support works, create four example\n repositories that can be used later as a submodule:\n \n -------------------------------------------------\n@@ -3914,7 +3914,7 @@ fact that such a commit brings together (\"merges\") two or more\n previous states represented by other commits.\n \n In other words, while a \"tree\" represents a particular directory state\n-of a working directory, a \"commit\" represents that state in \"time\",\n+of a working directory, a \"commit\" represents that state in time,\n and explains how we got there.\n \n You create a commit object by giving it the tree that describes the\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226029","messageId":"878772375.34519.1377626477170.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 06/11] Simplify \"How to make a commit\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:01:17Z","receivedAt":"2013-08-27T18:01:17Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nCombine the two cases for \"git add\" into one.\nAdd verb \"use\" to \"git rm\" case.\n\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 10 ++--------\n 1 file changed, 2 insertions(+), 8 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 465d9cb..98d2804 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1080,19 +1080,13 @@ produce no output at that point.\n \n Modifying the index is easy:\n \n-To update the index with the new contents of a modified file, use\n+To update the index with the contents of a new or modified file, use\n \n -------------------------------------------------\n $ git add path/to/file\n -------------------------------------------------\n \n-To add the contents of a new file to the index, use\n-\n--------------------------------------------------\n-$ git add path/to/file\n--------------------------------------------------\n-\n-To remove a file from the index and from the working tree,\n+To remove a file from the index and from the working tree, use\n \n -------------------------------------------------\n $ git rm path/to/file\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226030","messageId":"1088240617.34545.1377626528726.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 07/11] Improve section \"Manipulating branches\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:02:08Z","receivedAt":"2013-08-27T18:02:08Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nAdd some missing punctuation.\nSimplify description of \"git branch -d/-D\".\n\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 20 ++++++++------------\n 1 file changed, 8 insertions(+), 12 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 98d2804..c20e8df 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -268,27 +268,23 @@ Creating, deleting, and modifying branches is quick and easy; here's\n a summary of the commands:\n \n `git branch`::\n-\tlist all branches\n+\tlist all branches.\n `git branch `::\n \tcreate a new branch named ``, referencing the same\n-\tpoint in history as the current branch\n+\tpoint in history as the current branch.\n `git branch `::\n \tcreate a new branch named ``, referencing\n \t``, which may be specified any way you like,\n-\tincluding using a branch name or a tag name\n+\tincluding using a branch name or a tag name.\n `git branch -d `::\n-\tdelete the branch ``; if the branch you are deleting\n-\tpoints to a commit which is not reachable from the current\n-\tbranch, this command will fail with a warning.\n+\tdelete the branch ``; if the branch is not fully\n+\tmerged in its upstream branch or contained in the current branch, \n+\tthis command will fail with a warning.\n `git branch -D `::\n-\teven if the branch points to a commit not reachable\n-\tfrom the current branch, you may know that that commit\n-\tis still reachable from some other branch or tag. In that\n-\tcase it is safe to use this command to force Git to delete\n-\tthe branch.\n+\tdelete the branch `` irrespective of its merged status.\n `git checkout `::\n \tmake the current branch ``, updating the working\n-\tdirectory to reflect the version referenced by ``\n+\tdirectory to reflect the version referenced by ``.\n `git checkout -b `::\n \tcreate a new branch `` referencing ``, and\n \tcheck it out.\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226031","messageId":"1931551541.34581.1377626589383.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 08/11] Improve section \"Merging multiple trees\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:03:09Z","receivedAt":"2013-08-27T18:03:09Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nRemove unnecessary quoting.\nSimplify description of three-way merge.\n\nSigned-off-by: Jonathan Nieder \nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 27 +++++++++++++--------------\n 1 file changed, 13 insertions(+), 14 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex c20e8df..a7ca3e3 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -4004,27 +4004,26 @@ to see what the top commit was.\n Merging multiple trees\n ----------------------\n \n-Git helps you do a three-way merge, which you can expand to n-way by\n-repeating the merge procedure arbitrary times until you finally\n-\"commit\" the state. The normal situation is that you'd only do one\n-three-way merge (two parents), and commit it, but if you like to, you\n-can do multiple parents in one go.\n+Git can help you perform a three-way merge, which can in turn be\n+used for a many-way merge by repeating the merge procedure several\n+times. The usual situation is that you only do one three-way merge\n+(reconciling two lines of history) and commit the result, but if\n+you like to, you can merge several branches in one go.\n \n-To do a three-way merge, you need the two sets of \"commit\" objects\n-that you want to merge, use those to find the closest common parent (a\n-third \"commit\" object), and then use those commit objects to find the\n-state of the directory (\"tree\" object) at these points.\n+To perform a three-way merge, you start with the two commits you\n+want to merge, find their closest common parent (a third commit),\n+and compare the trees corresponding to these three commits.\n \n-To get the \"base\" for the merge, you first look up the common parent\n-of two commits with\n+To get the \"base\" for the merge, look up the common parent of two\n+commits:\n \n -------------------------------------------------\n $ git merge-base \n -------------------------------------------------\n \n-which will return you the commit they are both based on. You should\n-now look up the \"tree\" objects of those commits, which you can easily\n-do with (for example)\n+This prints the name of a commit they are both based on. You should\n+now look up the tree objects of those commits, which you can easily\n+do with\n \n -------------------------------------------------\n $ git cat-file commit | head -1\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226032","messageId":"2104057570.34585.1377626648812.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 09/11] Remove unnecessary historical note from \"Object storage format\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:04:08Z","receivedAt":"2013-08-27T18:04:08Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 2 --\n 1 file changed, 2 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex a7ca3e3..56bd088 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -4145,8 +4145,6 @@ about the data in the object. It's worth noting that the SHA-1 hash\n that is used to name the object is the hash of the original data\n plus this header, so `sha1sum` 'file' does not match the object name\n for 'file'.\n-(Historical note: in the dawn of the age of Git the hash\n-was the SHA-1 of the 'compressed' object.)\n \n As a result, the general consistency of an object can always be tested\n independently of the contents or the type of the object: all objects can\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226033","messageId":"240594913.34603.1377626698349.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 10/11] Remove irrelevant reference from \"Tying it all together\"","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:04:58Z","receivedAt":"2013-08-27T18:04:58Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\nSorry Jon, but this might not be of any help to new Git users ;)\n\nAcked-by: Jon Loeliger \nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 3 +--\n 1 file changed, 1 insertion(+), 2 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 56bd088..9149846 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -3924,8 +3924,7 @@ save the note about that state, in practice we tend to just write the\n result to the file pointed at by `.git/HEAD`, so that we can always see\n what the last committed state was.\n \n-Here is an ASCII art by Jon Loeliger that illustrates how\n-various pieces fit together.\n+Here is a picture that illustrates how various pieces fit together:\n \n ------------\n \n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226034","messageId":"860155028.34610.1377626750550.JavaMail.ngmail@webmail19.arcor-online.net","threadId":"34783","inReplyTo":"1403569571.34349.1377625974290.JavaMail.ngmail@webmail19.arcor-online.net","subject":"[PATCH 11/11] \"git prune\" is safe","fromName":"Thomas Ackermann","fromEmail":"th.acker@arcor.de","sentAt":"2013-08-27T18:05:50Z","receivedAt":"2013-08-27T18:05:50Z","isPatch":true,"sender":{"key":"th.acker@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1358536?v=4"},"body":"\n\"git prune\" is safe in case of concurrent accesses to a repository\nbut using it in such a case is not recommended.\n\nSigned-off-by: Thomas Ackermann \n---\n Documentation/user-manual.txt | 12 +++---------\n 1 file changed, 3 insertions(+), 9 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 9149846..ea843e6 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -3299,17 +3299,11 @@ state, you can just prune all unreachable objects:\n $ git prune\n ------------------------------------------------\n \n-and they'll be gone. But you should only run `git prune` on a quiescent\n+and they'll be gone. (You should only run `git prune` on a quiescent\n repository--it's kind of like doing a filesystem fsck recovery: you\n don't want to do that while the filesystem is mounted.\n-\n-(The same is true of `git fsck` itself, btw, but since\n-`git fsck` never actually *changes* the repository, it just reports\n-on what it found, `git fsck` itself is never 'dangerous' to run.\n-Running it while somebody is actually changing the repository can cause\n-confusing and scary messages, but it won't actually do anything bad. In\n-contrast, running `git prune` while somebody is actively changing the\n-repository is a *BAD* idea).\n+`git prune` is designed not to cause any harm in such cases of concurrent\n+accesses to a repository but you might receive confusing or scary messages.)\n \n [[recovering-from-repository-corruption]]\n Recovering from repository corruption\n-- \n1.8.3.msysgit.0\n\n\n---\nThomas\n"},{"id":"226038","messageId":"xmqqsixvc5wb.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1972849803.34382.1377626115372.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 01/11] Call it \"Git User Manual\" and remove reference to very old Git version","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:04:04Z","receivedAt":"2013-08-27T19:04:04Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Reviewed-by: Jonathan Nieder \n> Signed-off-by: Thomas Ackermann \n> ---\n\nI tend to agree with Jonathan that this should be \"User's Manual\".\n\nI've tentatively queued uncontroversial bits from v1 series already\non 'pu', by the way, and I do not think I have time to queue rerolls\nof any topic today, so please expect a longer turn-around time than\nusual.\n\n> Documentation/user-manual.txt | 5 ++---\n> 1 file changed, 2 insertions(+), 3 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index fe723e4..103ec9a 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -1,6 +1,5 @@\n> -Git User's Manual (for version 1.5.3 or newer)\n> -______________________________________________\n> -\n> +Git User Manual\n> +_______________\n> \n> Git is a fast distributed revision control system.\n"},{"id":"226039","messageId":"xmqqob8jc5s6.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1685937666.34465.1377626298173.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 04/11] Use \"git merge\" instead of \"git pull .\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:06:33Z","receivedAt":"2013-08-27T19:06:33Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> \"git pull .\" works, but \"git merge\" is the recommended\n> way for new users to do things. (The old description\n> also should have read \"The former is actually *not* very\n> commonly used\".)\n\nIt does not matter that you are unaware other people use it often.\nI'd suggest dropping the first hunk altogether.\n"},{"id":"226040","messageId":"xmqqk3j7c5ps.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"70864071.34489.1377626361141.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 05/11] Fix some typos and improve wording","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:07:59Z","receivedAt":"2013-08-27T19:07:59Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Signed-off-by: Thomas Ackermann \n> ---\n> Documentation/user-manual.txt | 20 ++++++++++----------\n> 1 file changed, 10 insertions(+), 10 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index 6241a43..465d9cb 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -219,7 +219,7 @@ of development leading to that point.\n> \n> The best way to see how this works is using the linkgit:gitk[1]\n> command; running gitk now on a Git repository and looking for merge\n> -commits will help understand how the Git organizes history.\n> +commits will help understand how Git organizes history.\n> \n> In the following, we say that commit X is \"reachable\" from commit Y\n> if commit X is an ancestor of commit Y. Equivalently, you could say\n> @@ -793,7 +793,7 @@ e05db0fd4f31dde7005f075a84f96b360d05984b\n> -------------------------------------------------\n> \n> Or you could recall that the `...` operator selects all commits\n> -contained reachable from either one reference or the other but not\n> +reachable from either one reference or the other but not\n> both; so\n> \n> -------------------------------------------------\n> @@ -820,7 +820,7 @@ You could just visually inspect the commits since e05db0fd:\n> $ gitk e05db0fd..\n> -------------------------------------------------\n> \n> -Or you can use linkgit:git-name-rev[1], which will give the commit a\n> +or you can use linkgit:git-name-rev[1], which will give the commit a\n\nI think I agree with Jonathan that this reads better with \"Or\", not\n\"or\".\n\nOther than that looks good to me. Thanks.\n"},{"id":"226041","messageId":"xmqqfvtvc5ou.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"878772375.34519.1377626477170.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 06/11] Simplify \"How to make a commit\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:08:33Z","receivedAt":"2013-08-27T19:08:33Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sounds good.\n"},{"id":"226043","messageId":"xmqqbo4jc5jn.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1088240617.34545.1377626528726.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 07/11] Improve section \"Manipulating branches\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:11:40Z","receivedAt":"2013-08-27T19:11:40Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> `git branch -d `::\n> -\tdelete the branch ``; if the branch you are deleting\n> -\tpoints to a commit which is not reachable from the current\n> -\tbranch, this command will fail with a warning.\n> +\tdelete the branch ``; if the branch is not fully\n> +\tmerged in its upstream branch or contained in the current branch, \n> +\tthis command will fail with a warning.\n\nThis is not a new problem, but it fails with an error, not a warning\n(which often is a message to caution but operation gets carried out\nanyway). For that matter, it might be better to say \"stops\", as it\nis not a failure but is saving the user from losing information (in\nother words, that is a different kind of success ;-).\n\nIt also stops you from deleting the branch you are currently on. I\nwonder if we want to mention that, too?\n\n> `git branch -D `::\n> -\teven if the branch points to a commit not reachable\n> -\tfrom the current branch, you may know that that commit\n> -\tis still reachable from some other branch or tag. In that\n> -\tcase it is safe to use this command to force Git to delete\n> -\tthe branch.\n> +\tdelete the branch `` irrespective of its merged status.\n> `git checkout `::\n> \tmake the current branch ``, updating the working\n> -\tdirectory to reflect the version referenced by ``\n> +\tdirectory to reflect the version referenced by ``.\n> `git checkout -b `::\n> \tcreate a new branch `` referencing ``, and\n> \tcheck it out.\n"},{"id":"226044","messageId":"xmqq7gf7c5ci.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1931551541.34581.1377626589383.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 08/11] Improve section \"Merging multiple trees\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:15:57Z","receivedAt":"2013-08-27T19:15:57Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Remove unnecessary quoting.\n> Simplify description of three-way merge.\n>\n> Signed-off-by: Jonathan Nieder \n> Signed-off-by: Thomas Ackermann \n> ---\n\nThe update is good, but I wonder if this belongs to \"User's manual\"\nthese days.\n\nPerhaps we should start thinking about ripping the \"how to use\nplumbing\" out of the end-user manual and moving them to the\ncore-tutorial where this kind of thing is more suited for.\n"},{"id":"226045","messageId":"xmqq38pvc5c1.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"2104057570.34585.1377626648812.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 09/11] Remove unnecessary historical note from \"Object storage format\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:16:14Z","receivedAt":"2013-08-27T19:16:14Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Good. Thanks.\n"},{"id":"226046","messageId":"xmqqy57naqp1.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"860155028.34610.1377626750550.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 11/11] \"git prune\" is safe","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:17:46Z","receivedAt":"2013-08-27T19:17:46Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> \"git prune\" is safe in case of concurrent accesses to a repository\n> but using it in such a case is not recommended.\n>\n> Signed-off-by: Thomas Ackermann \n> ---\n> Documentation/user-manual.txt | 12 +++---------\n> 1 file changed, 3 insertions(+), 9 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index 9149846..ea843e6 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -3299,17 +3299,11 @@ state, you can just prune all unreachable objects:\n> $ git prune\n> ------------------------------------------------\n> \n> -and they'll be gone. But you should only run `git prune` on a quiescent\n> +and they'll be gone. (You should only run `git prune` on a quiescent\n> repository--it's kind of like doing a filesystem fsck recovery: you\n> don't want to do that while the filesystem is mounted.\n> -\n> -(The same is true of `git fsck` itself, btw, but since\n> -`git fsck` never actually *changes* the repository, it just reports\n> -on what it found, `git fsck` itself is never 'dangerous' to run.\n> -Running it while somebody is actually changing the repository can cause\n> -confusing and scary messages, but it won't actually do anything bad. In\n> -contrast, running `git prune` while somebody is actively changing the\n> -repository is a *BAD* idea).\n> +`git prune` is designed not to cause any harm in such cases of concurrent\n> +accesses to a repository but you might receive confusing or scary messages.)\n\nThese new two lines are good, but did we lose the mention of \"fsck\"\nthat will report what is not dangling as dangling and such when run\nconcurrently with other operations? Is that intended?\n\n> \n> [[recovering-from-repository-corruption]]\n> Recovering from repository corruption\n"},{"id":"226049","messageId":"20130827194422.GW4110@google.com","threadId":"34783","inReplyTo":"xmqqob8jc5s6.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH 04/11] Use \"git merge\" instead of \"git pull .\"","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2013-08-27T19:44:22Z","receivedAt":"2013-08-27T19:44:22Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"On Tue, Aug 27, 2013 at 12:06:33PM -0700, Junio C Hamano wrote:\n> Thomas Ackermann writes:\n> \n> > \"git pull .\" works, but \"git merge\" is the recommended\n> > way for new users to do things. (The old description\n> > also should have read \"The former is actually *not* very\n> > commonly used\".)\n> \n> It does not matter that you are unaware other people use it often.\n> I'd suggest dropping the first hunk altogether.\n\nEh, the claim \"The former is actually very commonly used.\" is\nconfusing on its own (even though it used to be true) and elaborating\nwouldn't help much with education, so the first hunk makes sense to\nme. But maybe it should have been done in a separate patch. ;-)\n\nThanks,\nJonathan\n"},{"id":"226062","messageId":"xmqqfvtuc162.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"20130827194422.GW4110@google.com","subject":"Re: [PATCH 04/11] Use \"git merge\" instead of \"git pull .\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T20:46:13Z","receivedAt":"2013-08-27T20:46:13Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Nieder writes:\n\n> On Tue, Aug 27, 2013 at 12:06:33PM -0700, Junio C Hamano wrote:\n>> Thomas Ackermann writes:\n>> \n>> > \"git pull .\" works, but \"git merge\" is the recommended\n>> > way for new users to do things. (The old description\n>> > also should have read \"The former is actually *not* very\n>> > commonly used\".)\n>> \n>> It does not matter that you are unaware other people use it often.\n>> I'd suggest dropping the first hunk altogether.\n>\n> Eh, the claim \"The former is actually very commonly used.\" is\n> confusing on its own (even though it used to be true) and elaborating\n> wouldn't help much with education, so the first hunk makes sense to\n> me. But maybe it should have been done in a separate patch. ;-)\n\nYeah, it may make sense to replace it with something like \"... and\nif you think about the fact that your local repository is not at all\nspecial, it makes sense that you can pull from it just like you pull\nfrom other places\", without mentioning how common it is. I do agree\nthat it is a separate topic.\n"},{"id":"226065","messageId":"m3d2oybzr3.fsf@black.gullik.net","threadId":"34783","inReplyTo":"xmqqsixvc5wb.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH 01/11] Call it \"Git User Manual\" and remove reference to very old Git version","fromName":"Lars Gullik Bjønnes","fromEmail":"larsbj@gullik.org","sentAt":"2013-08-27T21:16:48Z","receivedAt":"2013-08-27T21:16:48Z","isPatch":true,"sender":{"key":"larsbj@gullik.org","avatar":"https://avatars.githubusercontent.com/u/15680184?v=4"},"body":"Junio C Hamano writes:\n\n| Thomas Ackermann writes:\n>\n>> Reviewed-by: Jonathan Nieder \n>> Signed-off-by: Thomas Ackermann \n>> ---\n>\n| I tend to agree with Jonathan that this should be \"User's Manual\".\n\nNot going into which one might be more correct, but a quick google\nsearch for \"User's Manual\" and \"User Manual\" give these results:\n\nUser's Manual: 5.7M hits.\nUser Manual: ~29M hits.\n\n-- \n\tLgb\n"},{"id":"226202","messageId":"xmqq8uzk5pt2.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1972849803.34382.1377626115372.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 01/11] Call it \"Git User Manual\" and remove reference to very old Git version","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-29T18:12:41Z","receivedAt":"2013-08-29T18:12:41Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Reviewed-by: Jonathan Nieder \n> Signed-off-by: Thomas Ackermann \n> ---\n> Documentation/user-manual.txt | 5 ++---\n> 1 file changed, 2 insertions(+), 3 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index fe723e4..103ec9a 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -1,6 +1,5 @@\n> -Git User's Manual (for version 1.5.3 or newer)\n> -______________________________________________\n> -\n> +Git User Manual\n> +_______________\n\nThe removal of the parenthesized part is good, but it should be\nUser's Manual, I think.\n"},{"id":"226203","messageId":"xmqq4na85psi.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1022906213.34401.1377626164272.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 02/11] Use current \"detached HEAD\" message","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-29T18:13:01Z","receivedAt":"2013-08-29T18:13:01Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Looks good; thanks.\n"},{"id":"226204","messageId":"xmqqzjs04b7g.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"1431109393.34423.1377626210746.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 03/11] Use current output for \"git repack\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-29T18:13:23Z","receivedAt":"2013-08-29T18:13:23Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Signed-off-by: Thomas Ackermann \n> ---\n> Documentation/user-manual.txt | 16 +++++++---------\n> 1 file changed, 7 insertions(+), 9 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index bdefd9a..3f44ca0 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -3203,17 +3203,15 @@ To put the loose objects into a pack, just run git repack:\n> \n> ------------------------------------------------\n> $ git repack\n> -Generating pack...\n> -Done counting 6020 objects.\n> -Deltifying 6020 objects.\n> - 100% (6020/6020) done\n> -Writing 6020 objects.\n> - 100% (6020/6020) done\n> -Total 6020, written 6020 (delta 4070), reused 0 (delta 0)\n> -Pack pack-3e54ad29d5b2e05838c75df582c65257b8d08e1c created.\n> +Counting objects: 6020, done.\n> +Delta compression using up to 4 threads.\n> +Compressing objects: 100% (6020/6020), done.\n> +Writing objects: 100% (6020/6020), done.\n> +Total 6020 (delta 4070), reused 0 (delta 0)\n> ------------------------------------------------\n> \n> -You can then run\n> +This creates a single \"pack file\" in .git/objects/pack/ \n> +containing all currently unpacked objects. You can then run\n> \n> ------------------------------------------------\n> $ git prune\n\nGood; thanks.\n"},{"id":"226208","messageId":"xmqqvc2o4b2k.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"70864071.34489.1377626361141.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 05/11] Fix some typos and improve wording","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-29T18:16:19Z","receivedAt":"2013-08-29T18:16:19Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Ackermann writes:\n\n> Signed-off-by: Thomas Ackermann \n> ---\n> Documentation/user-manual.txt | 20 ++++++++++----------\n> 1 file changed, 10 insertions(+), 10 deletions(-)\n>\n> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\n> index 6241a43..465d9cb 100644\n> --- a/Documentation/user-manual.txt\n> +++ b/Documentation/user-manual.txt\n> @@ -219,7 +219,7 @@ of development leading to that point.\n> \n> The best way to see how this works is using the linkgit:gitk[1]\n> command; running gitk now on a Git repository and looking for merge\n> -commits will help understand how the Git organizes history.\n> +commits will help understand how Git organizes history.\n> \n> In the following, we say that commit X is \"reachable\" from commit Y\n> if commit X is an ancestor of commit Y. Equivalently, you could say\n> @@ -793,7 +793,7 @@ e05db0fd4f31dde7005f075a84f96b360d05984b\n> -------------------------------------------------\n> \n> Or you could recall that the `...` operator selects all commits\n> -contained reachable from either one reference or the other but not\n> +reachable from either one reference or the other but not\n> both; so\n> \n> -------------------------------------------------\n> @@ -820,7 +820,7 @@ You could just visually inspect the commits since e05db0fd:\n> $ gitk e05db0fd..\n> -------------------------------------------------\n> \n> -Or you can use linkgit:git-name-rev[1], which will give the commit a\n> +or you can use linkgit:git-name-rev[1], which will give the commit a\n\nAs raised during the initial review, the previous sentence has ended\nand this begins a new sentence, I think, hence \"Or\", not \"or\".\n\n> name based on any tag it finds pointing to one of the commit's\n> descendants:\n> \n> @@ -864,8 +864,8 @@ because it outputs only commits that are not reachable from v1.5.0-rc1.\n> \n> As yet another alternative, the linkgit:git-show-branch[1] command lists\n> the commits reachable from its arguments with a display on the left-hand\n> -side that indicates which arguments that commit is reachable from. So,\n> -you can run something like\n> +side that indicates which arguments that commit is reachable from. \n> +So, if you run something like\n> -------------------------------------------------\n> $ git show-branch e05db0fd v1.5.0-rc0 v1.5.0-rc1 v1.5.0-rc2\n> @@ -877,15 +877,15 @@ available\n> ...\n> -------------------------------------------------\n> \n> -then search for a line that looks like\n> +then a line like\n> \n> -------------------------------------------------\n> + ++ [e05db0fd] Fix warnings in sha1_file.c - use C99 printf format if\n> available\n> -------------------------------------------------\n> \n> -Which shows that e05db0fd is reachable from itself, from v1.5.0-rc1, and\n> -from v1.5.0-rc2, but not from v1.5.0-rc0.\n> +shows that e05db0fd is reachable from itself, from v1.5.0-rc1,\n> +and from v1.5.0-rc2, and not from v1.5.0-rc0.\n\nThis is easier to read. Good.\n"},{"id":"226209","messageId":"xmqqr4dc4b1x.fsf@gitster.dls.corp.google.com","threadId":"34783","inReplyTo":"878772375.34519.1377626477170.JavaMail.ngmail@webmail19.arcor-online.net","subject":"Re: [PATCH 06/11] Simplify \"How to make a commit\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-29T18:16:42Z","receivedAt":"2013-08-29T18:16:42Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Looks good; thanks.\n"}]}