[PATCH] doc: fix a typo and clarify a sentence

2018-10-03 Thread Mihir Mehta
I noticed that git-merge-base was unlikely to actually be a git command,
and tried it in my shell. Seeing that it doesn't work, I cleaned up two
places in the docs where it appears.

Signed-off-by: Mihir Mehta 
---
 Documentation/git-diff.txt  | 5 +++--
 Documentation/howto/update-hook-example.txt | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt
index b180f1fa5..6173f569e 100644
--- a/Documentation/git-diff.txt
+++ b/Documentation/git-diff.txt
@@ -72,8 +72,9 @@ two blob objects, or changes between two files on disk.
This form is to view the changes on the branch containing
and up to the second , starting at a common ancestor
of both .  "git diff A\...B" is equivalent to
-   "git diff $(git-merge-base A B) B".  You can omit any one
-   of , which has the same effect as using HEAD instead.
+   "git diff $(git merge-base A B) B".  You can omit any one
+   of the two instances of , which has the same effect as
+   using HEAD in its place.
 
 Just in case if you are doing something exotic, it should be
 noted that all of the  in the above description, except
diff --git a/Documentation/howto/update-hook-example.txt 
b/Documentation/howto/update-hook-example.txt
index a5193b1e5..89821ec74 100644
--- a/Documentation/howto/update-hook-example.txt
+++ b/Documentation/howto/update-hook-example.txt
@@ -80,7 +80,7 @@ case "$1" in
   info "The branch '$1' is new..."
 else
   # updating -- make sure it is a fast-forward
-  mb=$(git-merge-base "$2" "$3")
+  mb=$(git merge-base "$2" "$3")
   case "$mb,$2" in
 "$2,$mb") info "Update is fast-forward" ;;
*)noff=y; info "This is not a fast-forward update.";;
-- 
2.19.0



[PATCH] doc: fix a typo and clarify a sentence

2018-10-10 Thread Mihir Mehta
I noticed that git-merge-base was unlikely to actually be a git command,
and tried it in my shell. Seeing that it doesn't work, I cleaned up two
places in the docs where it appears.

Signed-off-by: Mihir Mehta 
---
 Documentation/git-diff.txt  | 7 ---
 Documentation/howto/update-hook-example.txt | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt
index b180f1fa5..a122f9ded 100644
--- a/Documentation/git-diff.txt
+++ b/Documentation/git-diff.txt
@@ -72,10 +72,11 @@ two blob objects, or changes between two files on disk.
This form is to view the changes on the branch containing
and up to the second , starting at a common ancestor
of both .  "git diff A\...B" is equivalent to
-   "git diff $(git-merge-base A B) B".  You can omit any one
-   of , which has the same effect as using HEAD instead.
+   "git diff $(git merge-base A B) B".  You can omit any one
+   of the two instances of , which has the same effect as
+   using HEAD in its place; omitting both results in an empty diff.
 
-Just in case if you are doing something exotic, it should be
+Just in case you are doing something exotic, it should be
 noted that all of the  in the above description, except
 in the last two forms that use ".." notations, can be any
 .
diff --git a/Documentation/howto/update-hook-example.txt 
b/Documentation/howto/update-hook-example.txt
index a5193b1e5..89821ec74 100644
--- a/Documentation/howto/update-hook-example.txt
+++ b/Documentation/howto/update-hook-example.txt
@@ -80,7 +80,7 @@ case "$1" in
   info "The branch '$1' is new..."
 else
   # updating -- make sure it is a fast-forward
-  mb=$(git-merge-base "$2" "$3")
+  mb=$(git merge-base "$2" "$3")
   case "$mb,$2" in
 "$2,$mb") info "Update is fast-forward" ;;
*)noff=y; info "This is not a fast-forward update.";;
-- 
2.19.0



[no subject]

2018-10-10 Thread Mihir Mehta
Thanks, Junio. Instead of removing that part of the patch, I opted to
expand it to make it a little clearer (in my opinion) than it was
before. Let me know if this works.

Mihir.


[PATCH] doc: fix a typo and clarify a sentence

2018-10-10 Thread Mihir Mehta
I noticed that git-merge-base was unlikely to actually be a git command,
and tried it in my shell. Seeing that it doesn't work, I cleaned up two
places in the docs where it appears.

Signed-off-by: Mihir Mehta 
---
 Documentation/git-diff.txt  | 4 ++--
 Documentation/howto/update-hook-example.txt | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt
index b180f1fa5..030f162f3 100644
--- a/Documentation/git-diff.txt
+++ b/Documentation/git-diff.txt
@@ -72,10 +72,10 @@ two blob objects, or changes between two files on disk.
This form is to view the changes on the branch containing
and up to the second , starting at a common ancestor
of both .  "git diff A\...B" is equivalent to
-   "git diff $(git-merge-base A B) B".  You can omit any one
+   "git diff $(git merge-base A B) B".  You can omit any one
of , which has the same effect as using HEAD instead.
 
-Just in case if you are doing something exotic, it should be
+Just in case you are doing something exotic, it should be
 noted that all of the  in the above description, except
 in the last two forms that use ".." notations, can be any
 .
diff --git a/Documentation/howto/update-hook-example.txt 
b/Documentation/howto/update-hook-example.txt
index a5193b1e5..89821ec74 100644
--- a/Documentation/howto/update-hook-example.txt
+++ b/Documentation/howto/update-hook-example.txt
@@ -80,7 +80,7 @@ case "$1" in
   info "The branch '$1' is new..."
 else
   # updating -- make sure it is a fast-forward
-  mb=$(git-merge-base "$2" "$3")
+  mb=$(git merge-base "$2" "$3")
   case "$mb,$2" in
 "$2,$mb") info "Update is fast-forward" ;;
*)noff=y; info "This is not a fast-forward update.";;
-- 
2.19.0