Re: [PATCH 2/2] peel_onion(): teach $foo^{object} peeler

2013-04-02 Thread Michael Haggerty
While I was in the middle of suggesting documentation for this new syntax, I discovered that you already added documentation to your repo but didn't mention the new version on the mailing list (or maybe I overlooked it). It would be helpful if you would submit your own changes to the mailing list

Re: [PATCH 2/2] peel_onion(): teach $foo^{object} peeler

2013-04-02 Thread Junio C Hamano
Michael Haggerty mhag...@alum.mit.edu writes: Off topic: Your patch reminds me of something else that surprised me: there is no $userstring^{tag}. I suppose it would be a bit ambiguous, given that tags can point at tags, and it would also be less useful than the other suffixes. But its

Re: [PATCH 2/2] peel_onion(): teach $foo^{object} peeler

2013-04-02 Thread Michael Haggerty
On 04/02/2013 05:45 PM, Junio C Hamano wrote: Michael Haggerty mhag...@alum.mit.edu writes: Off topic: Your patch reminds me of something else that surprised me: there is no $userstring^{tag}. I suppose it would be a bit ambiguous, given that tags can point at tags, and it would also be

Re: [PATCH 2/2] peel_onion(): teach $foo^{object} peeler

2013-04-02 Thread Junio C Hamano
Michael Haggerty mhag...@alum.mit.edu writes: On 04/02/2013 05:45 PM, Junio C Hamano wrote: Also, v1.8.2^{tag} would be give the tag itself, while master^{tag} would not report the commit master but would error out, which would be useless. You are better off doing `git cat-file -t foo` and