# [PATCH] Documentation/git-am: Document supported --patch-format options

5 messages from 2014-03-11 to 2014-03-17. Participants: Chris Packham, Junio C Hamano.
Thread: https://gitlist.dev/t/36115

## Chris Packham, 2014-03-11 03:51

Subject: [PATCH] Documentation/git-am: Document supported --patch-format options
Message-ID: <1394509917-9679-1-git-send-email-judge.packham@gmail.com>
URL: https://gitlist.dev/e/1394509917-9679-1-git-send-email-judge.packham%40gmail.com

```
The --patch-format option has been supported for a while but it is not
mentioned in the man page and the short help cannot tell the user what
the supported formats are. Add the option to the man page along with the
supported options.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---
I've not bothered to actually explain what the options mean. I'm not
sure if readers will care aside from just trying them until one works
(that's all I did when I had a patch that failed detection).

 Documentation/git-am.txt |   12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt
index 54d8461..76bd359 100644
--- a/Documentation/git-am.txt
+++ b/Documentation/git-am.txt
@@ -12,9 +12,9 @@ SYNOPSIS
 'git am' [--signoff] [--keep] [--[no-]keep-cr] [--[no-]utf8]
 	 [--3way] [--interactive] [--committer-date-is-author-date]
 	 [--ignore-date] [--ignore-space-change | --ignore-whitespace]
-	 [--whitespace=<option>] [-C<n>] [-p<n>] [--directory=<dir>]
-	 [--exclude=<path>] [--include=<path>] [--reject] [-q | --quiet]
-	 [--[no-]scissors]
+	 [--whitespace=<option>] [-C<n>] [-p<n>] [--patch-format=<format>]
+	 [--directory=<dir>] [--exclude=<path>] [--include=<path>]
+	 [--reject] [-q | --quiet] [--[no-]scissors]
 	 [(<mbox> | <Maildir>)...]
 'git am' (--continue | --skip | --abort)
 
@@ -97,6 +97,12 @@ default.   You can use `--no-utf8` to override this.
 	program that applies
 	the patch.
 
+--patch-format::
+	By default the command will try to detect the patch format
+	automatically. This option allows the user to bypass the automatic
+	detection and specify the patch format that the patch(es) should be
+	intepreted as. Valid formats are mbox, stgit, stgit-series and hg.
+
 -i::
 --interactive::
 	Run interactively.
-- 
1.7.9.5

```

## Chris Packham, 2014-03-16 07:59

Subject: Re: [PATCH] Documentation/git-am: Document supported --patch-format options
Message-ID: <532559D2.3050802@gmail.com>
URL: https://gitlist.dev/e/532559D2.3050802%40gmail.com
In-Reply-To: <1394509917-9679-1-git-send-email-judge.packham@gmail.com>

```
On 11/03/14 16:51, Chris Packham wrote:
> The --patch-format option has been supported for a while but it is not
> mentioned in the man page and the short help cannot tell the user what
> the supported formats are. Add the option to the man page along with the
> supported options.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> I've not bothered to actually explain what the options mean. I'm not
> sure if readers will care aside from just trying them until one works
> (that's all I did when I had a patch that failed detection).
> 
>  Documentation/git-am.txt |   12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt
> index 54d8461..76bd359 100644
> --- a/Documentation/git-am.txt
> +++ b/Documentation/git-am.txt
> @@ -12,9 +12,9 @@ SYNOPSIS
>  'git am' [--signoff] [--keep] [--[no-]keep-cr] [--[no-]utf8]
>  	 [--3way] [--interactive] [--committer-date-is-author-date]
>  	 [--ignore-date] [--ignore-space-change | --ignore-whitespace]
> -	 [--whitespace=<option>] [-C<n>] [-p<n>] [--directory=<dir>]
> -	 [--exclude=<path>] [--include=<path>] [--reject] [-q | --quiet]
> -	 [--[no-]scissors]
> +	 [--whitespace=<option>] [-C<n>] [-p<n>] [--patch-format=<format>]
> +	 [--directory=<dir>] [--exclude=<path>] [--include=<path>]
> +	 [--reject] [-q | --quiet] [--[no-]scissors]
>  	 [(<mbox> | <Maildir>)...]
>  'git am' (--continue | --skip | --abort)
>  
> @@ -97,6 +97,12 @@ default.   You can use `--no-utf8` to override this.
>  	program that applies
>  	the patch.
>  
> +--patch-format::
> +	By default the command will try to detect the patch format
> +	automatically. This option allows the user to bypass the automatic
> +	detection and specify the patch format that the patch(es) should be
> +	intepreted as. Valid formats are mbox, stgit, stgit-series and hg.
> +
>  -i::
>  --interactive::
>  	Run interactively.
> 

Ping?

Actually now that I have the patch in a MUA I see the a simple
s/intepreted/interpreted/ fixup is required.

```

## Junio C Hamano, 2014-03-17 06:39

Subject: Re: [PATCH] Documentation/git-am: Document supported --patch-format options
Message-ID: <7vlhw92us1.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vlhw92us1.fsf%40alter.siamese.dyndns.org
In-Reply-To: <532559D2.3050802@gmail.com>

```
Chris Packham <judge.packham@gmail.com> writes:

> Ping?

Hasn't it been already cooking in 'next' for a few days?

```

## Chris Packham, 2014-03-17 07:35

Subject: Re: [PATCH] Documentation/git-am: Document supported --patch-format options
Message-ID: <5326A5DB.5070407@gmail.com>
URL: https://gitlist.dev/e/5326A5DB.5070407%40gmail.com
In-Reply-To: <7vlhw92us1.fsf@alter.siamese.dyndns.org>

```
On 17/03/14 19:39, Junio C Hamano wrote:
> Chris Packham <judge.packham@gmail.com> writes:
> 
>> Ping?
> 
> Hasn't it been already cooking in 'next' for a few days?
> 

Indeed I think I missed a "What's cooking". Do you want me to submit a
fixup for the spelling mistake?

```

## Chris Packham, 2014-03-17 07:50

Subject: [PATCH] fixup! Documentation/git-am: Document supported --patch-format options
Message-ID: <1395042649-16040-1-git-send-email-judge.packham@gmail.com>
URL: https://gitlist.dev/e/1395042649-16040-1-git-send-email-judge.packham%40gmail.com
In-Reply-To: <5326A5DB.5070407@gmail.com>

```
---
On 17/03/14 20:35, Chris Packham wrote:> On 17/03/14 19:39, Junio C Hamano wrote:
>> Chris Packham <judge.packham@gmail.com> writes:
>>
>>> Ping?
>>
>> Hasn't it been already cooking in 'next' for a few days?
>>
> 
> Indeed I think I missed a "What's cooking". Do you want me to submit a
> fixup for the spelling mistake?
> 

May as well just send it.

 Documentation/git-am.txt |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt
index 76bd359..f8a7d0c 100644
--- a/Documentation/git-am.txt
+++ b/Documentation/git-am.txt
@@ -101,7 +101,7 @@ default.   You can use `--no-utf8` to override this.
 	By default the command will try to detect the patch format
 	automatically. This option allows the user to bypass the automatic
 	detection and specify the patch format that the patch(es) should be
-	intepreted as. Valid formats are mbox, stgit, stgit-series and hg.
+	interpreted as. Valid formats are mbox, stgit, stgit-series and hg.
 
 -i::
 --interactive::
-- 
1.7.9.5

```