From: Erik Faye-Lund Date: Fri, 14 Sep 2012 17:11:34 GMT Subject: Re: [PATCH/RFC] test-lib: add support for colors without tput Message-ID: In-Reply-To: On Fri, Sep 14, 2012 at 7:08 PM, Elia Pinto wrote: > 2012/9/14 Erik Faye-Lund : >> On Fri, Sep 14, 2012 at 6:54 PM, Erik Faye-Lund wrote: >>> On Fri, Sep 14, 2012 at 6:41 PM, Erik Faye-Lund wrote: >>>> diff --git a/t/test-lib.sh b/t/test-lib.sh >>>> index 78c4286..7d1b34b 100644 >>>> --- a/t/test-lib.sh >>>> +++ b/t/test-lib.sh >>>> @@ -129,6 +129,20 @@ export _x05 _x40 _z40 LF >>>> # This test checks if command xyzzy does the right thing... >>>> # ' >>>> # . ./test-lib.sh >>>> + > Nice. But this setting should be check that we have a terminal first isn't ? > Some test like this before > > test "X$$TERM" != Xdumb \ > && test -t 1 2>/dev/null \ > && .... > > or the inverse logic. This is what automake and popt autogen.sh does. There's already such a check a few lines further down, and tput isn't used in such cases. -- *** Please reply-to-all at all times *** *** (do not pretend to know who is subscribed and who is not) *** *** Please avoid top-posting. *** The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free. You received this message because you are subscribed to the Google Groups "msysGit" group. To post to this group, send email to msysgit@googlegroups.com To unsubscribe from this group, send email to msysgit+unsubscribe@googlegroups.com For more options, and view previous threads, visit this group at http://groups.google.com/group/msysgit?hl=en_US?hl=en