Re: [PATCH 2/3] receive-pack: send pack-processing stderr over sideband

2012-09-21 Thread Jeff King
On Fri, Sep 21, 2012 at 10:25:24AM -0700, Junio C Hamano wrote: > > However, this does make things more consistent with upload-pack, which > > connects the stderr of pack-objects to sideband (which it must to handle > > progress). Furthermore, many of the messages from receive-pack are > > handled

Re: [PATCH 2/3] receive-pack: send pack-processing stderr over sideband

2012-09-21 Thread Junio C Hamano
Jeff King writes: > On Fri, Sep 21, 2012 at 09:49:40AM -0700, Junio C Hamano wrote: > >> > 2. No matter what the cause, we are probably better off >> > showing the errors to the client. If the client and the >> > server admin are not the same entity, it is probably >> > much easi

Re: [PATCH 2/3] receive-pack: send pack-processing stderr over sideband

2012-09-21 Thread Jeff King
On Fri, Sep 21, 2012 at 09:49:40AM -0700, Junio C Hamano wrote: > > 2. No matter what the cause, we are probably better off > > showing the errors to the client. If the client and the > > server admin are not the same entity, it is probably > > much easier for the client to cut-an

Re: [PATCH 2/3] receive-pack: send pack-processing stderr over sideband

2012-09-21 Thread Junio C Hamano
Jeff King writes: > Receive-pack invokes either unpack-objects or index-pack to > handle the incoming pack. However, we do not redirect the > stderr of the sub-processes at all, so it is never seen by > the client. From the initial thread adding sideband support, > which is here: > > http://thr

[PATCH 2/3] receive-pack: send pack-processing stderr over sideband

2012-09-20 Thread Jeff King
Receive-pack invokes either unpack-objects or index-pack to handle the incoming pack. However, we do not redirect the stderr of the sub-processes at all, so it is never seen by the client. From the initial thread adding sideband support, which is here: http://thread.gmane.org/gmane.comp.version-