Am 29.02.2016 um 22:19 schrieb Junio C Hamano:
Stefan Beller <[email protected]> writes:

Maybe we want to remove the struct child_process from the
function signature of the callbacks and callbacks need to rely on
the data provided solely thru the pointer as passed around for
callback purposes, which the user is free to use for any kind
of data.

I think that is the most sensible.

I also think that is the better approach. Dumping out the argv array is not the best end-user experience. It is just a debugging aid, and for that we have (or should extend if necessary) GIT_TRACE infrastructure. Moreover, a command that failed should have printed error messages, and it is not necessary to follow it up with another "A child process exited with code N" message.

-- Hannes

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to