# [PATCH] t/README: fix typo

6 messages from 2022-01-04 to 2022-01-06. Participants: Marc Strapetz via GitGitGadget, Johannes Schindelin, Marc Strapetz, Junio C Hamano.
Thread: https://gitlist.dev/t/57184

## Marc Strapetz via GitGitGadget, 2022-01-04 23:17

Subject: [PATCH] t/README: fix typo
Message-ID: <pull.1110.git.1641338224631.gitgitgadget@gmail.com>
URL: https://gitlist.dev/e/pull.1110.git.1641338224631.gitgitgadget%40gmail.com

```
From: Marc Strapetz <marc.strapetz@syntevo.com>

Signed-off-by: Marc Strapetz <marc.strapetz@syntevo.com>
---
    t/README: fix typo

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1110%2Fmstrap%2Ffeature%2FREADME-typo-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1110/mstrap/feature/README-typo-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1110

 t/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/README b/t/README
index 2353a4c5e13..f48e0542cdc 100644
--- a/t/README
+++ b/t/README
@@ -466,7 +466,7 @@ explicitly providing repositories when accessing submodule objects is
 complete or needs to be abandoned for whatever reason (in which case the
 migrated codepaths still retain their performance benefits).
 
-GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space speparated list of
+GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space separated list of
 prereqs that are required to succeed. If a prereq in this list is triggered by
 a test and then fails then the whole test run will abort. This can help to make
 sure the expected tests are executed and not silently skipped when their

base-commit: dcc0cd074f0c639a0df20461a301af6d45bd582e
-- 
gitgitgadget

```

## Johannes Schindelin, 2022-01-05 16:22

Subject: Re: [PATCH] t/README: fix typo
Message-ID: <nycvar.QRO.7.76.6.2201051701120.7076@tvgsbejvaqbjf.bet>
URL: https://gitlist.dev/e/nycvar.QRO.7.76.6.2201051701120.7076%40tvgsbejvaqbjf.bet
In-Reply-To: <pull.1110.git.1641338224631.gitgitgadget@gmail.com>

```
Hi Marc,

On Tue, 4 Jan 2022, Marc Strapetz via GitGitGadget wrote:

> From: Marc Strapetz <marc.strapetz@syntevo.com>
>
> Signed-off-by: Marc Strapetz <marc.strapetz@syntevo.com>
> ---
>     t/README: fix typo
>
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1110%2Fmstrap%2Ffeature%2FREADME-typo-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1110/mstrap/feature/README-typo-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/1110
>
>  t/README | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/README b/t/README
> index 2353a4c5e13..f48e0542cdc 100644
> --- a/t/README
> +++ b/t/README
> @@ -466,7 +466,7 @@ explicitly providing repositories when accessing submodule objects is
>  complete or needs to be abandoned for whatever reason (in which case the
>  migrated codepaths still retain their performance benefits).
>
> -GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space speparated list of
> +GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space separated list of

Good find!

I'm curious, how did you find it? If it was a spell checker, did it also
find the "dotsh" typo in the same file (it should be "dots", I think)?

Ciao,
Dscho

>  prereqs that are required to succeed. If a prereq in this list is triggered by
>  a test and then fails then the whole test run will abort. This can help to make
>  sure the expected tests are executed and not silently skipped when their
>
> base-commit: dcc0cd074f0c639a0df20461a301af6d45bd582e
> --
> gitgitgadget
>

```

## Marc Strapetz, 2022-01-05 16:47

Subject: Re: [PATCH] t/README: fix typo
Message-ID: <9f74fc04-ea68-6bdd-3341-ecf7a2aed6cd@syntevo.com>
URL: https://gitlist.dev/e/9f74fc04-ea68-6bdd-3341-ecf7a2aed6cd%40syntevo.com
In-Reply-To: <nycvar.QRO.7.76.6.2201051701120.7076@tvgsbejvaqbjf.bet>

```
Hi Dscho,

On 05/01/2022 17:22, Johannes Schindelin wrote:
>> @@ -466,7 +466,7 @@ explicitly providing repositories when accessing submodule objects is
>>   complete or needs to be abandoned for whatever reason (in which case the
>>   migrated codepaths still retain their performance benefits).
>>
>> -GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space speparated list of
>> +GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space separated list of
> 
> Good find!
> 
> I'm curious, how did you find it? If it was a spell checker, did it also

I found it by plain reading :) as I had troubles to get a test working.

 > find the "dotsh" typo in the same file (it should be "dots", I think)?

I wouldn't have noticed that, but I agree that it should be 
"test_name_without_dots".

-Marc

```

## Junio C Hamano, 2022-01-05 20:41

Subject: Re: [PATCH] t/README: fix typo
Message-ID: <xmqqo84phq3x.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqqo84phq3x.fsf%40gitster.g
In-Reply-To: <pull.1110.git.1641338224631.gitgitgadget@gmail.com>

```
"Marc Strapetz via GitGitGadget" <gitgitgadget@gmail.com> writes:

> -GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space speparated list of
> +GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space separated list of

Thanks, queued.

```

## Marc Strapetz, 2022-01-06 06:17

Subject: Re: [PATCH] t/README: fix typo
Message-ID: <b265de46-9ccd-4f31-d51c-1df1b39606d5@syntevo.com>
URL: https://gitlist.dev/e/b265de46-9ccd-4f31-d51c-1df1b39606d5%40syntevo.com
In-Reply-To: <9f74fc04-ea68-6bdd-3341-ecf7a2aed6cd@syntevo.com>

```
On 05/01/2022 17:47, Marc Strapetz wrote:
> Hi Dscho,
> 
> On 05/01/2022 17:22, Johannes Schindelin wrote:
>>> @@ -466,7 +466,7 @@ explicitly providing repositories when accessing 
>>> submodule objects is
>>>   complete or needs to be abandoned for whatever reason (in which 
>>> case the
>>>   migrated codepaths still retain their performance benefits).
>>>
>>> -GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space speparated 
>>> list of
>>> +GIT_TEST_REQUIRE_PREREQ=<list> allows specifying a space separated 
>>> list of
>>
>> Good find!
>>
>> I'm curious, how did you find it? If it was a spell checker, did it also
> 
> I found it by plain reading :) as I had troubles to get a test working.
> 
>  > find the "dotsh" typo in the same file (it should be "dots", I think)?
> 
> I wouldn't have noticed that, but I agree that it should be 
> "test_name_without_dots".

Please disregard my last statement. "test_name_without_dotsh" means 
"test name without .sh extension". For 
"t2109-update-index-index-version.sh", the directory will be "trash 
directory.t2109-update-index-index-version".

-Marc

```

## Junio C Hamano, 2022-01-06 17:48

Subject: Re: [PATCH] t/README: fix typo
Message-ID: <xmqqwnjceox1.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqqwnjceox1.fsf%40gitster.g
In-Reply-To: <b265de46-9ccd-4f31-d51c-1df1b39606d5@syntevo.com>

```
Marc Strapetz <marc.strapetz@syntevo.com> writes:

>>  > find the "dotsh" typo in the same file (it should be "dots", I
>> think)?
>> I wouldn't have noticed that, but I agree that it should be 
>> "test_name_without_dots".
>
> Please disregard my last statement. "test_name_without_dotsh" means
> "test name without .sh extension". For 
> "t2109-update-index-index-version.sh", the directory will be "trash
> directory.t2109-update-index-index-version".

Yup.  "without_dot_sh" might have been a better way to spell it, but
"without_dotsh" certainly is better than "without_dots".

Thanks for being extra careful.  In any case, the original fix is
good and has already been queued.


```
