Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-10-08 Thread Junio C Hamano
Mike Hommey writes: > After refreshing the patch against current "next", it appears that > there is such a distinction: > > $ ./git-log --notes=fdsfgsfdg HEAD^! --pretty=short > warning: notes ref refs/notes/fdsfgsfdg is invalid > commit e5b68b2e879608d881c2e3600ce84962fcdefc88 > Author: Mike Hom

Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-10-07 Thread Mike Hommey
On Mon, Jul 13, 2015 at 09:35:40AM -0700, Junio C Hamano wrote: > What is interesting is to think what should happen when amlog@{1} > is given. The version with your patch gives the same output as we > saw earlier, because amlog@{1} tree exists, but does not know about > your patch (yet), so does

Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-07-13 Thread Junio C Hamano
Mike Hommey writes: > init_notes() is the main point of entry to the notes API. It is an arbitrary > restriction that all it allows as input is a strict ref name, when callers > may want to give an arbitrary treeish. > > However, some operations that require updating the notes tree require a > st

Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-07-13 Thread Junio C Hamano
Mike Hommey writes: >> - Make sure that we show "there is no such tree-ish, no way to look >>up any note to any commit from there" and "I understood the tree >>you gave me, but there is no note for that commit" differently. > > How would you reconcile that with the usual "there are only

Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-07-13 Thread Mike Hommey
On Mon, Jul 13, 2015 at 09:35:40AM -0700, Junio C Hamano wrote: > Mike Hommey writes: > > > init_notes() is the main point of entry to the notes API. It is an arbitrary > > restriction that all it allows as input is a strict ref name, when callers > > may want to give an arbitrary treeish. > > >

Re: [PATCH] notes: Allow treeish expressions as notes ref

2015-07-13 Thread Junio C Hamano
Mike Hommey writes: > init_notes() is the main point of entry to the notes API. It is an arbitrary > restriction that all it allows as input is a strict ref name, when callers > may want to give an arbitrary treeish. > > However, some operations that require updating the notes tree require a > st

[PATCH] notes: Allow treeish expressions as notes ref

2015-07-10 Thread Mike Hommey
init_notes() is the main point of entry to the notes API. It is an arbitrary restriction that all it allows as input is a strict ref name, when callers may want to give an arbitrary treeish. However, some operations that require updating the notes tree require a strict ref name, because they would