[PATCH 1/2] refs: disallow ref components starting with hyphen

2012-07-16 Thread Michael Schubert
Currently, we allow refname components to start with a hyphen. There's no good reason to do so and it troubles the parseopt infrastructure. Explicitly refuse refname components starting with a hyphen inside check_refname_component(). Revert 63486240, which is obsolete now. Signed-off-by: Michael

Re: [PATCH 1/2] refs: disallow ref components starting with hyphen

2012-07-16 Thread Junio C Hamano
Michael Schubert msc...@elegosoft.com writes: Currently, we allow refname components to start with a hyphen. There's no good reason to do so... That is way too weak as a justification to potentially break existing repositories. Refusal upon attempted creation is probably OK, which is why the

Re: [PATCH 1/2] refs: disallow ref components starting with hyphen

2012-07-16 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: Michael Schubert msc...@elegosoft.com writes: Currently, we allow refname components to start with a hyphen. There's no good reason to do so... That is way too weak as a justification to potentially break existing repositories. Refusal upon