Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-21 Thread Jeff King
On Mon, Jan 21, 2013 at 03:33:46PM -0800, Junio C Hamano wrote: > Jeff King writes: > > >> It may even make sense to have "git push" honor it, excluding them > >> from "git push --mirror" (or "git push --all" if some of the > >> branches are hidden); I haven't thought things through, though. > >

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-21 Thread Junio C Hamano
Jeff King writes: >> It may even make sense to have "git push" honor it, excluding them >> from "git push --mirror" (or "git push --all" if some of the >> branches are hidden); I haven't thought things through, though. > > That is harder, as that is something that happens on the client. How > doe

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-21 Thread Jeff King
On Sun, Jan 20, 2013 at 10:06:33AM -0800, Junio C Hamano wrote: > Jeff King writes: > > >>[uploadPack] > >>hiderefs = refs/changes > > > > Would you want to do the same thing on receive-pack? It could benefit > > from the same reduction in network cost (although it tends to be in

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-21 Thread Jeff King
On Sun, Jan 20, 2013 at 02:08:32PM -0800, Junio C Hamano wrote: > Junio C Hamano writes: > > > Jeff King writes: > > > >>> [uploadPack] > >>> hiderefs = refs/changes > >> > >> Would you want to do the same thing on receive-pack? It could benefit > >> from the same reduction in netwo

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-21 Thread Jeff King
On Sat, Jan 19, 2013 at 11:16:00AM -0800, Junio C Hamano wrote: > For example, if you mirror-clone from either of my repositories from > GitHub: > > git clone --mirror git://github.com/git/git/ > git clone --mirror git://github.com/gitster/git/ > > you will see stuff that does not belong

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-20 Thread Duy Nguyen
On Sun, Jan 20, 2013 at 2:16 AM, Junio C Hamano wrote: > Duy Nguyen writes: > >> Should the client side learn how to list hidden refs too? I'm thinking >> of an extreme case where upload-pack advertises nothing (or maybe just >> refs/heads/master) and it's up to the client to ask for the ref >> s

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-20 Thread Junio C Hamano
Junio C Hamano writes: > Jeff King writes: > >>> [uploadPack] >>> hiderefs = refs/changes >> >> Would you want to do the same thing on receive-pack? It could benefit >> from the same reduction in network cost (although it tends to be invoked >> less frequently than upload-pack).

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-20 Thread Junio C Hamano
Junio C Hamano writes: > Duy Nguyen writes: > >> Should the client side learn how to list hidden refs too? I'm thinking >> of an extreme case where upload-pack advertises nothing (or maybe just >> refs/heads/master) and it's up to the client to ask for the ref >> selection it's interested in. up

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-20 Thread Junio C Hamano
Jeff King writes: >> [uploadPack] >> hiderefs = refs/changes > > Would you want to do the same thing on receive-pack? It could benefit > from the same reduction in network cost (although it tends to be invoked > less frequently than upload-pack). Do *I* want to do? Not when th

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-19 Thread Junio C Hamano
Duy Nguyen writes: > Should the client side learn how to list hidden refs too? I'm thinking > of an extreme case where upload-pack advertises nothing (or maybe just > refs/heads/master) and it's up to the client to ask for the ref > selection it's interested in. upload-pack may need more updates

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-19 Thread Jeff King
On Fri, Jan 18, 2013 at 04:37:04PM -0800, Junio C Hamano wrote: > This is an early preview of reducing the network cost while talking > with a repository with tons of refs, most of which are of use by > very narrow audiences (e.g. refs under Gerrit's refs/changes/ are > useful only for people who

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-18 Thread Michael Haggerty
On 01/19/2013 01:37 AM, Junio C Hamano wrote: > This is an early preview of reducing the network cost while talking > with a repository with tons of refs, most of which are of use by > very narrow audiences (e.g. refs under Gerrit's refs/changes/ are > useful only for people who are interested in t

Re: [PATCH 0/2] Hiding some refs in ls-remote

2013-01-18 Thread Duy Nguyen
On Sat, Jan 19, 2013 at 7:37 AM, Junio C Hamano wrote: > This is an early preview of reducing the network cost while talking > with a repository with tons of refs, most of which are of use by > very narrow audiences (e.g. refs under Gerrit's refs/changes/ are > useful only for people who are inter

[PATCH 0/2] Hiding some refs in ls-remote

2013-01-18 Thread Junio C Hamano
This is an early preview of reducing the network cost while talking with a repository with tons of refs, most of which are of use by very narrow audiences (e.g. refs under Gerrit's refs/changes/ are useful only for people who are interested in the changes under review). As long as these narrow aud