Thread (1 message) 1 message, 1 author, 2017-06-26

Re: [PATCH v6 1/6] t0021: keep filter log files on comparison

From: Junio C Hamano <hidden>
Date: 2017-06-26 17:31:50

Lars Schneider [off-list ref] writes:
quoted
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help