Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info

2013-05-19 Thread Felipe Contreras
On Sun, May 19, 2013 at 6:48 AM, Ramkumar Ramachandra wrote: > Duy Nguyen wrote: >> Exactly. I already explained why %(upstream) can't be used in 00/09. >> "tracking" may not be perfect. Somebody might want >> "tracking:upstream:short". It does not look quite nice. > > Which is why I suggested kee

Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info

2013-05-19 Thread Ramkumar Ramachandra
Duy Nguyen wrote: > Exactly. I already explained why %(upstream) can't be used in 00/09. > "tracking" may not be perfect. Somebody might want > "tracking:upstream:short". It does not look quite nice. Which is why I suggested keeping upstream, upstream:short, and introducing upstream:diff and upstr

Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info

2013-05-19 Thread Duy Nguyen
On Sun, May 19, 2013 at 6:38 PM, Felipe Contreras wrote: > On Sun, May 19, 2013 at 6:18 AM, Ramkumar Ramachandra > wrote: >> Nguyễn Thái Ngọc Duy wrote: >>> diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c >>> index 498d703..b10d48a 100644 >>> --- a/builtin/for-each-ref.c >>> +++ b/bu

Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info

2013-05-19 Thread Felipe Contreras
On Sun, May 19, 2013 at 6:18 AM, Ramkumar Ramachandra wrote: > Nguyễn Thái Ngọc Duy wrote: >> diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c >> index 498d703..b10d48a 100644 >> --- a/builtin/for-each-ref.c >> +++ b/builtin/for-each-ref.c >> @@ -76,6 +76,8 @@ static struct { >>

Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info

2013-05-19 Thread Ramkumar Ramachandra
Nguyễn Thái Ngọc Duy wrote: > diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c > index 498d703..b10d48a 100644 > --- a/builtin/for-each-ref.c > +++ b/builtin/for-each-ref.c > @@ -76,6 +76,8 @@ static struct { > { "symref" }, > { "flag" }, > { "current" }, > +