From: Junio C Hamano <gitster@pobox.com>
To: Lars Schneider <larsxschneider@gmail.com>
Cc: "Git Users" <git@vger.kernel.org>, "Jeff King" <peff@peff.net>,
"Torsten Bögershausen" <tboegi@web.de>, "Eric Wong" <e@80x24.org>,
"Taylor Blau" <ttaylorr@github.com>,
peartben@gmail.com
Subject: Re: [PATCH v6 1/6] t0021: keep filter log files on comparison
Date: Mon, 26 Jun 2017 10:31:43 -0700 [thread overview]
Message-ID: <xmqqvani64g0.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <79C07E47-251D-4869-8EEA-11A9E336ABF3@gmail.com> (Lars Schneider's message of "Mon, 26 Jun 2017 11:02:06 +0200")
Lars Schneider <larsxschneider@gmail.com> writes:
>> It would become a problem _if_ we want future users of this helper
>> to reuse the same expect (or actual) multiple times and start from
>> an unmodified one. There may be some other reason why you do not
>> want the comparison to smudge these files. Please state what that
>> reason is before saying "fix this".
>
> Understood. How about this?
>
> The filter log files are modified on comparison. That might be
> unexpected by the caller. It would be even undesirable if the caller
> wants to reuse the original log files.
>
> Address these issues by using temp files for modifications. This is
> useful for the subsequent patch 'convert: add "status=delayed" to
> filter process protocol'.
The updated one is much more understandable. Thanks.
> If this is OK, then do you want me to resend the series or can you fix it
> in place?
In general, I am OK running "rebase -i" to polish the log message
unless there are other changes to the patches planned.
next prev parent reply other threads:[~2017-06-26 17:31 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-25 18:21 [PATCH v6 0/6] convert: add "status=delayed" to filter process protocol Lars Schneider
2017-06-25 18:21 ` [PATCH v6 1/6] t0021: keep filter log files on comparison Lars Schneider
2017-06-25 22:12 ` Junio C Hamano
2017-06-26 9:02 ` Lars Schneider
2017-06-26 17:31 ` Junio C Hamano [this message]
2017-06-25 18:21 ` [PATCH v6 2/6] t0021: make debug log file name configurable Lars Schneider
2017-06-25 22:15 ` Junio C Hamano
2017-06-25 18:21 ` [PATCH v6 3/6] t0021: write "OUT" only on success Lars Schneider
2017-06-25 22:17 ` Junio C Hamano
2017-06-26 9:26 ` Lars Schneider
2017-06-26 17:50 ` Junio C Hamano
2017-06-26 18:32 ` Lars Schneider
2017-06-25 18:21 ` [PATCH v6 4/6] convert: put the flags field before the flag itself for consistent style Lars Schneider
2017-06-25 22:19 ` Junio C Hamano
2017-06-25 18:21 ` [PATCH v6 5/6] convert: move multiple file filter error handling to separate function Lars Schneider
2017-06-26 17:56 ` Junio C Hamano
2017-06-27 2:51 ` Stefan Beller
2017-06-26 18:00 ` Junio C Hamano
2017-06-25 18:21 ` [PATCH v6 6/6] convert: add "status=delayed" to filter process protocol Lars Schneider
2017-06-26 19:02 ` Junio C Hamano
2017-06-26 21:28 ` Lars Schneider
2017-06-26 22:13 ` Junio C Hamano
2017-06-26 22:38 ` Junio C Hamano
2017-06-27 12:07 ` Lars Schneider
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqvani64g0.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox.com \
--cc=e@80x24.org \
--cc=git@vger.kernel.org \
--cc=larsxschneider@gmail.com \
--cc=peartben@gmail.com \
--cc=peff@peff.net \
--cc=tboegi@web.de \
--cc=ttaylorr@github.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.