Re: [PATCH] t1400: drop debug `echo` to actually execute `test`

From

Jeff King <peff@peff.net>

Date

Sep 29, 2018, 08:00 UTC

Message-ID

<20180929080016.GH2174@sigill.intra.peff.net>

In-Reply-To

<20180928154359.26919-1-martin.agren@gmail.com>

On Fri, Sep 28, 2018 at 05:43:59PM +0200, Martin Ågren wrote:

> Instead of running `test "foo" = "$(bar)"`, we prefix the whole thing
> with `echo`. Comparing to nearby tests makes it clear that this is just
> debug leftover. This line has actually been modified four times since it
> was introduced in e52290428b (General ref log reading improvements.,
> 2006-05-19) and the `echo` has always survived. Let's finally drop it.

Hmm, yeah. I cannot see how this echo was ever accomplishing anything.

> This script could need some more cleanups. This is just an immediate fix
> so that we actually test what we intend to.

> ---
>  t/t1400-update-ref.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

The patch looks good to me. Thanks!

-Peff