Re: [PATCH 3/9] ref-filter: add support for %(path) atom
From: Matthieu Moy <hidden>
Date: 2016-06-15 23:06:45
Junio C Hamano [off-list ref] writes:
Matthieu Moy [off-list ref] writes:quoted
quoted
quoted
quoted
This adds %(path) and %(path:short) atoms. The %(path) atom will print the path of the given ref, while %(path:short) will only print the subdirectory of the given ref.What does "path" mean in this context? How is it different from %(refname)? I found the answer below, but I could not guess from the doc and commit message. Actually, I'm not sure %(path) is the right name. If you want the "file/directory" analogy, then %(dirname) would be better.Noted will change.Note: I don't completely like %(dirname) either. I'm convinced it's better than %(path), but there may be a better option.Is that a derived form of the refname, just like %(refname:short) that is 'master' for a ref whose %(refname) is 'refs/heads/master' is a derived form of %(refname), and ":short" is what tells the formatting machinery what kind of derivation is desired? If so would %(refname:dir) & %(refname:base) be more in line with the overall structure?
Yes, indeed much better. It's still about the refnames, so a specialized version of %(refname) makes much more sense than a new atom. -- Matthieu Moy http://www-verimag.imag.fr/~moy/