# [PATCH] Documentation/git-rebase: clarify -s option

2 messages from 2010-11-12 to 2010-11-13. Participants: Martin von Zweigbergk, Junio C Hamano.
Thread: https://gitlist.dev/t/25732

## Martin von Zweigbergk, 2010-11-12 18:57

Subject: [PATCH] Documentation/git-rebase: clarify -s option
Message-ID: <1289588269-6920-1-git-send-email-martin.von.zweigbergk@gmail.com>
URL: https://gitlist.dev/e/1289588269-6920-1-git-send-email-martin.von.zweigbergk%40gmail.com

```
Clarify that it is '-s' itself that implies '--merge'. Currently it may
seem like '--merge' is implied "If there is no `-s` option".

Signed-off-by: Martin von Zweigbergk <martin.von.zweigbergk@gmail.com>
---
 Documentation/git-rebase.txt |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
index 30e5c0e..f3753a8 100644
--- a/Documentation/git-rebase.txt
+++ b/Documentation/git-rebase.txt
@@ -244,9 +244,9 @@ other words, the sides are swapped.
 
 -s <strategy>::
 --strategy=<strategy>::
-	Use the given merge strategy.
+	Use the given merge strategy.  This implies --merge.
 	If there is no `-s` option 'git merge-recursive' is used
-	instead.  This implies --merge.
+	instead.
 +
 Because 'git rebase' replays each commit from the working branch
 on top of the <upstream> branch using the given strategy, using
-- 
1.7.3.2.167.ga361b

```

## Junio C Hamano, 2010-11-13 01:59

Subject: Re: [PATCH] Documentation/git-rebase: clarify -s option
Message-ID: <7vtyjmdlup.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vtyjmdlup.fsf%40alter.siamese.dyndns.org
In-Reply-To: <1289588269-6920-1-git-send-email-martin.von.zweigbergk@gmail.com>

```
Martin von Zweigbergk <martin.von.zweigbergk@gmail.com> writes:

> Clarify that it is '-s' itself that implies '--merge'. Currently it may
> seem like '--merge' is implied "If there is no `-s` option".
>
> Signed-off-by: Martin von Zweigbergk <martin.von.zweigbergk@gmail.com>
> ---
>  Documentation/git-rebase.txt |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
> index 30e5c0e..f3753a8 100644
> --- a/Documentation/git-rebase.txt
> +++ b/Documentation/git-rebase.txt
> @@ -244,9 +244,9 @@ other words, the sides are swapped.
>  
>  -s <strategy>::
>  --strategy=<strategy>::
> -	Use the given merge strategy.
> +	Use the given merge strategy.  This implies --merge.
>  	If there is no `-s` option 'git merge-recursive' is used
> -	instead.  This implies --merge.
> +	instead.

The first change is absolutely right---thanks for starting this.  However,
"if there is no `-s`..." describes the default when `--merge` alone is
given, so these last two lines may need to move elsewhere, no?

```