Re: [PATCH] alias: detect loops in mixed execution mode
- From
Jeff King <peff@peff.net>
- Date
- Oct 22, 2018, 21:15 UTC
- Message-ID
- <20181022211545.GA15713@sigill.intra.peff.net>
- In-Reply-To
- <87efckctqa.fsf@evledraar.gmail.com>
On Sat, Oct 20, 2018 at 09:18:21PM +0200, Ævar Arnfjörð Bjarmason wrote:
Show 5 quoted lines
> We didn't support chained aliases at all before, so I think the odds > that people will run into this now will increase as they add "!" to > existing aliases, and I'd like to have git's UI friendly enough to tell > users what went wrong by default, and not have to resort to the likes of > GIT_TRACE=1 which really should be left to powerusers.
It's true that non-! aliases couldn't recurse before, but couldn't "!" ones always do so?
-Peff