Re: [PATCH v4 3/3] t4113: put executable lines to test_expect_success

From: Shuqi Liang <cheskaqiqi@gmail.com>
Date: Feb 7, 2023, 19:55 UTC
Message-ID: <CAMO4yUFMTSMuyDjiM_Ssa7X95Cm2x_uUR73Uf1UJoaPxqvoELg@mail.gmail.com>
In-Reply-To: <230207.86h6vx51x3.gmgdl@evledraar.gmail.com>

Hi Ævar,

On Tue, Feb 7, 2023 at 3:12 AM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 6 quoted lines

> The error doesn't tell us much, instead of "make prove" or "prove
> <name>" running e.g.:
>
>         ./t4113-apply-ending.sh -vixd
>
> Gives you better output.

Thanks, this is really helpful.

Show 8 quoted lines

> But this is almost certainly that you're trying to insert leading
> whitespace into a line that's in a <<-EOF here-doc, the "-" part of that
> means that your leading whitespace is being stripped.
>
> A typical idiom for that is have a marker for the start of line, and
> strip the whitespace with "sed". See this for existing examples:
>
>         git grep 'sed.*\^.*>.*EOF'

Thank you for the tip! I'll try to fix the problem as soon as possible.

--------- Thanks, Shuqi