Re: [PATCH v6 05/16] Prepare the builtins for a libified merge_recursive()

2 messages, 2 authors, 2016-08-02 · open the first message on its own page

Re: [PATCH v6 05/16] Prepare the builtins for a libified merge_recursive()

From: Junio C Hamano <hidden>
Date: 2016-08-01 19:14:55

Johannes Schindelin [off-list ref] writes:
Previously, callers of merge_trees() or merge_recursive() expected that
code to die() with an error message. This used to be okay because we
called those commands from scripts, and had a chance to print out a
message in case the command failed fatally (read: with exit code 128).

As scripting incurs its own set of problems (portability, speed,
idiosynchracies of different shells, limited data structures leading to
I think I typofixed this when I queued the previous one on 'pu'
already, but s/synch/sync/; 

Re: [PATCH v6 05/16] Prepare the builtins for a libified merge_recursive()

From: Johannes Schindelin <hidden>
Date: 2016-08-02 08:10:32

Hi Junio,

On Mon, 1 Aug 2016, Junio C Hamano wrote:
Johannes Schindelin [off-list ref] writes:
quoted
Previously, callers of merge_trees() or merge_recursive() expected that
code to die() with an error message. This used to be okay because we
called those commands from scripts, and had a chance to print out a
message in case the command failed fatally (read: with exit code 128).

As scripting incurs its own set of problems (portability, speed,
idiosynchracies of different shells, limited data structures leading to
I think I typofixed this when I queued the previous one on 'pu'
already, but s/synch/sync/; 
Whoops. Fixed locally.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help