Re: [PATCH 07/11] Improve section "Manipulating branches"

2013-08-27 Thread Junio C Hamano
Thomas Ackermann  writes:

>  `git branch -d `::
> - delete the branch ``; if the branch you are deleting
> - points to a commit which is not reachable from the current
> - branch, this command will fail with a warning.
> + delete the branch ``; if the branch is not fully
> + merged in its upstream branch or contained in the current branch, 
> + this command will fail with a warning.

This is not a new problem, but it fails with an error, not a warning
(which often is a message to caution but operation gets carried out
anyway).  For that matter, it might be better to say "stops", as it
is not a failure but is saving the user from losing information (in
other words, that is a different kind of success ;-).

It also stops you from deleting the branch you are currently on.  I
wonder if we want to mention that, too?

>  `git branch -D `::
> - even if the branch points to a commit not reachable
> - from the current branch, you may know that that commit
> - is still reachable from some other branch or tag.  In that
> - case it is safe to use this command to force Git to delete
> - the branch.
> + delete the branch `` irrespective of its merged status.
>  `git checkout `::
>   make the current branch ``, updating the working
> - directory to reflect the version referenced by ``
> + directory to reflect the version referenced by ``.
>  `git checkout -b  `::
>   create a new branch `` referencing ``, and
>   check it out.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 07/11] Improve section "Manipulating branches"

2013-08-27 Thread Thomas Ackermann

Add some missing punctuation.
Simplify description of "git branch -d/-D".

Signed-off-by: Thomas Ackermann 
---
 Documentation/user-manual.txt | 20 
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt
index 98d2804..c20e8df 100644
--- a/Documentation/user-manual.txt
+++ b/Documentation/user-manual.txt
@@ -268,27 +268,23 @@ Creating, deleting, and modifying branches is quick and 
easy; here's
 a summary of the commands:
 
 `git branch`::
-   list all branches
+   list all branches.
 `git branch `::
create a new branch named ``, referencing the same
-   point in history as the current branch
+   point in history as the current branch.
 `git branch  `::
create a new branch named ``, referencing
``, which may be specified any way you like,
-   including using a branch name or a tag name
+   including using a branch name or a tag name.
 `git branch -d `::
-   delete the branch ``; if the branch you are deleting
-   points to a commit which is not reachable from the current
-   branch, this command will fail with a warning.
+   delete the branch ``; if the branch is not fully
+   merged in its upstream branch or contained in the current branch, 
+   this command will fail with a warning.
 `git branch -D `::
-   even if the branch points to a commit not reachable
-   from the current branch, you may know that that commit
-   is still reachable from some other branch or tag.  In that
-   case it is safe to use this command to force Git to delete
-   the branch.
+   delete the branch `` irrespective of its merged status.
 `git checkout `::
make the current branch ``, updating the working
-   directory to reflect the version referenced by ``
+   directory to reflect the version referenced by ``.
 `git checkout -b  `::
create a new branch `` referencing ``, and
check it out.
-- 
1.8.3.msysgit.0


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