Re: [PATCH v2 2/2] object name: introduce '^{/!-<negativepattern>}' notation
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 11, 2016, 17:13 UTC
- Message-ID
- <xmqqpox8xcn6.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <CACsJy8Dar9sCkTg_SQsDUOWYNQ1PHjmA0KcgrvpvmeY=yVXPMg@mail.gmail.com>
Duy Nguyen <pclouds@gmail.com> writes:
Show 8 quoted lines
> On Fri, Jan 8, 2016 at 1:04 PM, Stephen Smith <ischis2@cox.net> wrote:
>>> > +test_expect_success 'ref^{/!-}' '
>>> > + test_must_fail git rev-parse master^{/!-}
>>> > +'
>
> Shouldn't it be ^{!/... instead of ^{/!... ? People could have a
> pattern starting with "!" and /! will change its meaning. On the other
> hand, anything else after { is still reserved and can safely be used.http://thread.gmane.org/gmane.comp.version-control.git/40460/focus=40477 clarifies the above, I think.
Back then we seem to have been thinking only about ":" as the overall "search the history to name an object traversing from all tips" prefix, but the current one is described in terms of the more generally useful "<committish>^{/<pattern>}" syntax and the escape hatch applies--they are both handled by get_sha1_oneline() the same way.