Re: Symbolic refs break ref advertisement on 1.8.4.3+

2013-11-18 Thread Junio C Hamano
Jeff King writes: > On Sun, Nov 17, 2013 at 01:39:52AM +1100, Bryan Turner wrote: > >> Aphrael:example bturner$ for ((i=1;i<21;i++)); do git symbolic-ref >> refs/heads/syms/$i refs/heads/master; done >> Aphrael:example bturner$ git ls-remote . >> fatal: protocol error: impossibly long line >> fat

Re: Symbolic refs break ref advertisement on 1.8.4.3+

2013-11-17 Thread Jeff King
On Sun, Nov 17, 2013 at 01:39:52AM +1100, Bryan Turner wrote: > Aphrael:example bturner$ for ((i=1;i<21;i++)); do git symbolic-ref > refs/heads/syms/$i refs/heads/master; done > Aphrael:example bturner$ git ls-remote . > fatal: protocol error: impossibly long line > fatal: Could not read from remo

Symbolic refs break ref advertisement on 1.8.4.3+

2013-11-16 Thread Bryan Turner
According to a git bisect between the v1.8.4 and v1.8.4.3 tags, it appears the changes in 5e7dcad, "upload-pack: send non-HEAD symbolic refs", cause the ref advertisement to fail of the repository has more than a handful of symbolic refs. Here's a simple reproduce case (tested on Bash): Aphrael:ex