On 9/24/2021 22:49, Jeff King wrote:
OK, that makes things a bit easier. The colors, as you noticed, can be disabled by config. The other thing you're seeing is ANSI ESC[K, which is used to clear to the end of line. We use this in a couple places, notably when relaying progress lines from the server (with the "remote:" prefix) which may use carriage-returns to overwrite lines.
Those would be some of the culprits. I'll have to do a 'script' and see what it is spitting out.
Anyway, there's no config option to disable that. However, we do disable it if TERM is empty or set to "dumb" (and instead just write some extra spaces to clear out the line). So that may be an option, though of course setting TERM=dumb may affect other programs you use.
Editors, in particular, tend not to like interacting with TERM=dumb as they have no idea how to behave around one ("I need CE and UP" comes to mind).
I don't think it would be unreasonable to have a config option to select whether we use the ANSI or dumb-term version.
GIT_TERM might be an option to override TERM, but I am loath to actually suggest YAEV.
Nah, it sounds like you actually set the variables correctly. We've just assumed that we can get by with ANSI codes as a lowest common denominator in the modern world, without having to resort to all the complexities of using a terminfo library. It's worked pretty well so far. ;)
Laughing out loud at that. Part of me is apologetic to be The Weird Kid. The other part of me is looking for more ways to be weird. Thank you for taking a look at this.
-Peff
Cheers, --*greywolf;
