On Sun, Feb 14, 2010 at 11:04:13PM -0600, Jonathan Nieder wrote: > Jeff King wrote: > > On Sun, Feb 14, 2010 at 09:25:33PM -0600, Jonathan Nieder wrote: > > >> + if tty <&1 >/dev/null 2>/dev/null > > > > We seem to use "test -t 1" for this elsewhere, and I don't see us using > > "tty" anywhere else. It is POSIX, and I tested that even Solaris 8 and > > AIX 6.1 have it. So I don't think it is a portability issue, but others > > may know more than I. > > tty is portable, but "test -t 1" is cleaner. Thanks for the pointer. There is another one in the test script in patch 6/6, btw. -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