Re: [PATCH] object-name: fix reversed ordering with magic pathspecs

From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Dec 6, 2024, 12:25 UTC
Message-ID: <cfc7adaa-ccf3-4f27-807c-2eed70e56aa8@app.fastmail.com>
In-Reply-To: <xmqqcyi5vxbj.fsf@gitster.g>

On Fri, Dec 6, 2024, at 12:57, Junio C Hamano wrote:

> [snip]
>     Recently it was reported [*1*] that "look for the youngest
>     reachable commit with log message that match the given pattern"
>     syntax (e.g. :/myMessagePattern, or HEAD^{/myMessagePattern})

But these are slightly different since `:/` searches in all refs. They are only equivalent (i.e. with Patrick’s fix) when you have just made a commit (like in Patrick’s test).

And the longer syntax doesn’t seem to be affected by any regressions. In this repo:

    $ # wrong
    $ git log --format=reference -1 :/Merge
    3857284f7b8 (Merge refs/heads/master from ., 2005-08-24)
    $ # correct (looks correct)
    $ git log --format=reference -1 HEAD^{/Merge}
    4a611ee7ebd (Merge branch 'kn/ref-transaction-hook-with-reflog', 2024-11-27)

>     started to show older commit.