On Mon, Dec 15, 2008 at 12:15:38AM -0800, Junio C Hamano wrote: > I haven't heard anything more about this, but if you were indeed > discussing the change made by 61b8050, I think the fix should just be like > this. > [...] > - dup2(pager_process.in, 2); > + if (isatty(2)) > + dup2(pager_process.in, 2); I can't speak for the original poster, but I do think this behavior should be less surprising to users who perform the one particular redirection (and in the other 99% of cases, should behave exactly the same). So I think it's a positive change. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html