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

On Sun, May 19, 2013 at 6:38 PM, Felipe Contreras
[off-list ref] wrote:

On Sun, May 19, 2013 at 6:18 AM, Ramkumar Ramachandra
[off-list ref] wrote:

quoted

Nguyễn Thái Ngọc Duy wrote:

quoted

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" },
+       { "tracking" },
+       { "tracking:upstream" },
 };

You just threw the upstream atom (and "upstream:short") out the window :|

Huh? Those don't print the tracking information, do they?
"tracking:upstream" prints the upstream, but other things as well I
suppose.

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.
--
Duy