Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-23 Thread Carlos Martín Nieto
On Wed, 2013-11-06 at 15:42 -0800, Shawn Pearce wrote: > On Wed, Nov 6, 2013 at 1:41 PM, Carlos Martín Nieto wrote: > > On Wed, 2013-11-06 at 12:32 -0800, Junio C Hamano wrote: > >> I'll queue these for now, but I doubt the wisdom of this series, > >> given that the ship has already sailed long ti

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Shawn Pearce
On Wed, Nov 6, 2013 at 2:54 PM, Jeff King wrote: > If we instead introduced "no-thin", it is more like: > > 1. Receive-pack starts advertising "no-thin" (as dictated by > circumstances, as Carlos describes). > > 2. Send-pack which does not understand no-thin will ignore it and send >

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Shawn Pearce
On Wed, Nov 6, 2013 at 1:41 PM, Carlos Martín Nieto wrote: > On Wed, 2013-11-06 at 12:32 -0800, Junio C Hamano wrote: >> I'll queue these for now, but I doubt the wisdom of this series, >> given that the ship has already sailed long time ago. >> >> Currently, no third-party implementation of a rec

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Jeff King
On Wed, Nov 06, 2013 at 02:25:50PM -0800, Junio C Hamano wrote: > > Support in the code isn't always enough. The particular case that > > brought this on is one where the index-pack implementation can deal with > > thin packs just fine. > > > > This particular service takes the pack which the clie

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Junio C Hamano
Carlos Martín Nieto writes: > On Wed, 2013-11-06 at 12:32 -0800, Junio C Hamano wrote: >> I'll queue these for now, but I doubt the wisdom of this series, >> given that the ship has already sailed long time ago. >> >> Currently, no third-party implementation of a receiving end can >> accept thin

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Carlos Martín Nieto
On Wed, 2013-11-06 at 12:32 -0800, Junio C Hamano wrote: > I'll queue these for now, but I doubt the wisdom of this series, > given that the ship has already sailed long time ago. > > Currently, no third-party implementation of a receiving end can > accept thin push, because "thin push" is not a c

Re: [PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Junio C Hamano
Carlos Martín Nieto writes: > Hi all, > > This comes as a result of the discussion starting at [0] about > git-push assuming that a server will always support thin packs. Most > out there in fact do, but this isn't necessarily the case. > > Some implementations may not have support for it yet, or

[PATCH 0/2] thin-pack capability for send-pack/receive-pack

2013-11-06 Thread Carlos Martín Nieto
Hi all, This comes as a result of the discussion starting at [0] about git-push assuming that a server will always support thin packs. Most out there in fact do, but this isn't necessarily the case. Some implementations may not have support for it yet, or the server might be running in an environ