On Thu, Sep 11, 2025 at 04:32:56PM -0400, Jeff King wrote: > On Thu, Sep 11, 2025 at 07:31:04PM +0200, Kristoffer Haugsbakk wrote: > > > But it turns out there is a regression in my code with the recursion > > detection. Compare: > > > > $ ./git -c alias.one=two -c alias.two=two one -h > > 'one' is aliased to 'two' > > 'two' is aliased to 'two' > > fatal: recursive alias: two > > Your example there is a little funny; "two" is recursive to itself, even > without "one". We detect that case in handle_alias(), and we would still > detect: Also, I meant to say: good catch. :) I did not even think about loop detection at all in my suggestions. -Peff
