Re: [PATCH 5/9] for-each-ref: add %(tracking[:upstream]) for tracking info
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- May 19, 2013, 12:03 UTC
- Message-ID
- <CAMP44s0_o6xgkbo3rT1rXSRS6kmasiooVohZcO3W+PcspbSz0w@mail.gmail.com>
- In-Reply-To
- <CALkWK0m8HbYRAJdPgN7CnX_VMTX36GG-=BcqM+NTKwxX_0xqzg@mail.gmail.com>
On Sun, May 19, 2013 at 6:48 AM, Ramkumar Ramachandra <artagnon@gmail.com> wrote:
Show 8 quoted lines
> 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 upstream:shortdiff (or :tracking if you > prefer that) in [0/9].
Yeah, but there won't be any upstream in %(tracking). Besides, if we manage to get downstream, we could do %(tracking:downstream). I think %(tracking) and %(short:tracking) make sense.
-- Felipe Contreras