Re: [PATCH v4 5/5] fetch: reduce duplicate in ref update status lines with placeholder

2016-06-27 Thread Junio C Hamano
Eric Sunshine writes: > In my brain, at least, '$' is associated so strongly with regex that > "origin/$" is interpreted automatically as anchoring "origin/" at the > end of string, and "refs/$/head" just feels weird. > > On the other hand, given the familiarity of shell globbing, "origin/*" > an

Re: [PATCH v4 5/5] fetch: reduce duplicate in ref update status lines with placeholder

2016-06-26 Thread Duy Nguyen
On Mon, Jun 27, 2016 at 6:33 AM, Eric Sunshine wrote: > On Sun, Jun 26, 2016 at 1:58 AM, Nguyễn Thái Ngọc Duy > wrote: >> In the "remote -> local" line, if either ref is a substring of the >> other, the common part in the other string is replaced with "$". For >> example >> >> abc

Re: [PATCH v4 5/5] fetch: reduce duplicate in ref update status lines with placeholder

2016-06-26 Thread Eric Sunshine
On Sun, Jun 26, 2016 at 1:58 AM, Nguyễn Thái Ngọc Duy wrote: > In the "remote -> local" line, if either ref is a substring of the > other, the common part in the other string is replaced with "$". For > example > > abc-> origin/abc > refs/pull/123/head -> pull/123 > > becom

[PATCH v4 5/5] fetch: reduce duplicate in ref update status lines with placeholder

2016-06-25 Thread Nguyễn Thái Ngọc Duy
In the "remote -> local" line, if either ref is a substring of the other, the common part in the other string is replaced with "$". For example abc-> origin/abc refs/pull/123/head -> pull/123 become abc -> origin/$ refs/$/head -> pull/123 Activated with f