# [PATCH] doc: fix form -> from typo

6 messages from 2019-06-25 to 2019-06-26. Participants: Catalin Criste, Martin Ågren, Johannes Schindelin, Junio C Hamano, Thomas Gummerer.
Thread: https://gitlist.dev/t/51377

## Catalin Criste, 2019-06-25 06:36

Subject: [PATCH] doc: fix form -> from typo
Message-ID: <0102016b8d597569-c1f6cfdc-cb45-4428-8737-cb1bc30655d8-000000@eu-west-1.amazonses.com>
URL: https://gitlist.dev/e/0102016b8d597569-c1f6cfdc-cb45-4428-8737-cb1bc30655d8-000000%40eu-west-1.amazonses.com

```
Signed-off-by: Catalin Criste <cris_linu_w@yahoo.com>
---
 Documentation/git-stash.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt
index 7ef8c4791177b..b96ce2d02d149 100644
--- a/Documentation/git-stash.txt
+++ b/Documentation/git-stash.txt
@@ -88,7 +88,7 @@ save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q
 
 	This option is deprecated in favour of 'git stash push'.  It
 	differs from "stash push" in that it cannot take pathspecs,
-	and any non-option arguments form the message.
+	and any non-option arguments from the message.
 
 list [<options>]::
 

--
https://github.com/git/git/pull/521

```

## Martin Ågren, 2019-06-25 09:41

Subject: Re: [PATCH] doc: fix form -> from typo
Message-ID: <CAN0heSraZh+j04qjeaVtS5bsNoE=Hf_FBU-kfcB+69BkZz+zHg@mail.gmail.com>
URL: https://gitlist.dev/e/CAN0heSraZh%2Bj04qjeaVtS5bsNoE%3DHf_FBU-kfcB%2B69BkZz%2BzHg%40mail.gmail.com
In-Reply-To: <0102016b8d597569-c1f6cfdc-cb45-4428-8737-cb1bc30655d8-000000@eu-west-1.amazonses.com>

```
Hi Catalin

Welcome to the list!

On Tue, 25 Jun 2019 at 09:43, Catalin Criste <cris_linu_w@yahoo.com> wrote:

> @@ -88,7 +88,7 @@ save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q
>
>         This option is deprecated in favour of 'git stash push'.  It
>         differs from "stash push" in that it cannot take pathspecs,
> -       and any non-option arguments form the message.
> +       and any non-option arguments from the message.

I think this is actually intended as "form". It took me a couple of
readings, but what this paragraph wants to say is that any non-option
arguments will be used to form (construct) the message.

Do you have any suggestions as to how this could be made clearer?
There are at least two of us that have stumbled on this. :-)

Cheers
Martin

```

## Johannes Schindelin, 2019-06-25 11:40

Subject: Re: [PATCH] doc: fix form -> from typo
Message-ID: <nycvar.QRO.7.76.6.1906251339050.44@tvgsbejvaqbjf.bet>
URL: https://gitlist.dev/e/nycvar.QRO.7.76.6.1906251339050.44%40tvgsbejvaqbjf.bet
In-Reply-To: <CAN0heSraZh+j04qjeaVtS5bsNoE=Hf_FBU-kfcB+69BkZz+zHg@mail.gmail.com>

```
Hi Martin,

On Tue, 25 Jun 2019, Martin Ågren wrote:

> On Tue, 25 Jun 2019 at 09:43, Catalin Criste <cris_linu_w@yahoo.com> wrote:
>
> > @@ -88,7 +88,7 @@ save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q
> >
> >         This option is deprecated in favour of 'git stash push'.  It
> >         differs from "stash push" in that it cannot take pathspecs,
> > -       and any non-option arguments form the message.
> > +       and any non-option arguments from the message.
>
> I think this is actually intended as "form". It took me a couple of
> readings, but what this paragraph wants to say is that any non-option
> arguments will be used to form (construct) the message.

You're right! I totally misread this, too.

> Do you have any suggestions as to how this could be made clearer?
> There are at least two of us that have stumbled on this. :-)

Make that three.

Maybe something like

	This option is deprecated in favour of 'git stash push'.  It
	differs from "stash push" in that it cannot take pathspecs;
	Instead, all non-option arguments are concatenated to form the stash
	message.

?

Ciao,
Dscho

```

## Martin Ågren, 2019-06-25 17:49

Subject: Re: [PATCH] doc: fix form -> from typo
Message-ID: <CAN0heSr=E6yHkUHiGE=nezB72+P_R2iSZcYKDZHp1JTPj4f2QA@mail.gmail.com>
URL: https://gitlist.dev/e/CAN0heSr%3DE6yHkUHiGE%3DnezB72%2BP_R2iSZcYKDZHp1JTPj4f2QA%40mail.gmail.com
In-Reply-To: <nycvar.QRO.7.76.6.1906251339050.44@tvgsbejvaqbjf.bet>

```
On Tue, 25 Jun 2019 at 13:40, Johannes Schindelin
<Johannes.Schindelin@gmx.de> wrote:
> On Tue, 25 Jun 2019, Martin Ågren wrote:

> > Do you have any suggestions as to how this could be made clearer?
> > There are at least two of us that have stumbled on this. :-)
>
> Make that three.
>
> Maybe something like
>
>         This option is deprecated in favour of 'git stash push'.  It
>         differs from "stash push" in that it cannot take pathspecs;
>         Instead, all non-option arguments are concatenated to form the stash
>         message.
>
> ?

Looks good. I would probably avoid the semicolon ";" and in this case
just replace it with a period. But it's nothing I feel strongly about.

Martin

```

## Junio C Hamano, 2019-06-25 20:07

Subject: Re: [PATCH] doc: fix form -> from typo
Message-ID: <xmqqv9wte8ev.fsf@gitster-ct.c.googlers.com>
URL: https://gitlist.dev/e/xmqqv9wte8ev.fsf%40gitster-ct.c.googlers.com
In-Reply-To: <CAN0heSraZh+j04qjeaVtS5bsNoE=Hf_FBU-kfcB+69BkZz+zHg@mail.gmail.com>

```
Martin Ågren <martin.agren@gmail.com> writes:

> Hi Catalin
>
> Welcome to the list!
>
> On Tue, 25 Jun 2019 at 09:43, Catalin Criste <cris_linu_w@yahoo.com> wrote:
>
>> @@ -88,7 +88,7 @@ save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q
>>
>>         This option is deprecated in favour of 'git stash push'.  It
>>         differs from "stash push" in that it cannot take pathspecs,
>> -       and any non-option arguments form the message.
>> +       and any non-option arguments from the message.
>
> I think this is actually intended as "form". It took me a couple of
> readings, but what this paragraph wants to say is that any non-option
> arguments will be used to form (construct) the message.
>
> Do you have any suggestions as to how this could be made clearer?
> There are at least two of us that have stumbled on this. :-)

"become"?

```

## Thomas Gummerer, 2019-06-26 08:22

Subject: Re: [PATCH] doc: fix form -> from typo
Message-ID: <20190626082234.GC2423@hank.intra.tgummerer.com>
URL: https://gitlist.dev/e/20190626082234.GC2423%40hank.intra.tgummerer.com
In-Reply-To: <CAN0heSraZh+j04qjeaVtS5bsNoE=Hf_FBU-kfcB+69BkZz+zHg@mail.gmail.com>

```
On 06/25, Martin Ågren wrote:
> Hi Catalin
> 
> Welcome to the list!
> 
> On Tue, 25 Jun 2019 at 09:43, Catalin Criste <cris_linu_w@yahoo.com> wrote:
> 
> > @@ -88,7 +88,7 @@ save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q
> >
> >         This option is deprecated in favour of 'git stash push'.  It
> >         differs from "stash push" in that it cannot take pathspecs,
> > -       and any non-option arguments form the message.
> > +       and any non-option arguments from the message.
> 
> I think this is actually intended as "form". It took me a couple of
> readings, but what this paragraph wants to say is that any non-option
> arguments will be used to form (construct) the message.
> 
> Do you have any suggestions as to how this could be made clearer?
> There are at least two of us that have stumbled on this. :-)

Even though I originally wrote this I had to have a double take on it.
Maybe what you're saying above would be good?  "and any non-option
arguments are used to form the message" sounds a little clearer to
me.  Or maybe even use the word construct?  Dunno.

```