Re: [PATCH v3 3/8] upload/receive-pack: allow hiding ref hierarchies

2013-02-06 Thread Jeff King
On Tue, Feb 05, 2013 at 07:45:01AM -0800, Junio C Hamano wrote: In the earlier review, I mentioned making this per-service, but I see that is not the case here. Do you have an argument against doing so? Perhaps then I misunderstood your intention. By reminding me of the receive-pack

Re: [PATCH v3 3/8] upload/receive-pack: allow hiding ref hierarchies

2013-02-06 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Tue, Feb 05, 2013 at 07:45:01AM -0800, Junio C Hamano wrote: In the earlier review, I mentioned making this per-service, but I see that is not the case here. Do you have an argument against doing so? Perhaps then I misunderstood your intention. By

Re: [PATCH v3 3/8] upload/receive-pack: allow hiding ref hierarchies

2013-02-05 Thread Jeff King
On Wed, Jan 30, 2013 at 10:45:37AM -0800, Junio C Hamano wrote: Teach upload-pack and receive-pack to omit some refs from their initial advertisements by paying attention to the transfer.hiderefs multi-valued configuration variable. Any ref that is under the hierarchies listed on the value

Re: [PATCH v3 3/8] upload/receive-pack: allow hiding ref hierarchies

2013-02-05 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Wed, Jan 30, 2013 at 10:45:37AM -0800, Junio C Hamano wrote: Teach upload-pack and receive-pack to omit some refs from their initial advertisements by paying attention to the transfer.hiderefs multi-valued configuration variable. Any ref that is under the

[PATCH v3 3/8] upload/receive-pack: allow hiding ref hierarchies

2013-01-30 Thread Junio C Hamano
Teach upload-pack and receive-pack to omit some refs from their initial advertisements by paying attention to the transfer.hiderefs multi-valued configuration variable. Any ref that is under the hierarchies listed on the value of this variable is excluded from responses to requests made by