On Fri, 9 Jul 2021 at 20:21, Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Avishay Matayev <me@xxxxxxxxxxx> writes: > > > On Wed, 7 Jul 2021 at 01:03, Junio C Hamano <gitster@xxxxxxxxx> wrote: > >> > >> Avishay Matayev <me@xxxxxxxxxxx> writes: > >> > >> > There are several behaviors where git will not attempt to do an action if a > >> > tty is not present, for example - `git_pager` (pager.c) will instruct git > >> > to not use the GIT_PAGER environment variable if stdout is not a pty. > >> > >> In general, I am negative on this approach, even though I agree that > >> the issue you are trying to solve is worth solving. "Force pretend > >> that we are talking to a TTY" exposes too much of the internal > >> logic in the implementation to the end users. > >> > >> Often we use isatty() as an approximation for "are we being > >> interactive?", and I wouldn't have as much problem as I have with > >> the "FORCE_TTY" to an approach to give users a knob to tell us "do > >> not use your logic to guess---I am telling you that we are > >> interactive, so behave as such". > > I'm not sure that I understand your view on this patch. > > `git_isatty()` does exactly that, it gives the users that knob, doesn't it? > > > > If you don't find the approach in this patch good enough, what exactly > > do you suggest? > > Start with not tying this feature with the word "TTY", probably. Well then, what are other indicators of 'interactivity' besides "TTY"?
