2012/9/14 Elia Pinto <gitter.spiros@xxxxxxxxx>: > 2012/9/14 Erik Faye-Lund <kusmabite@xxxxxxxxx>: >> On Fri, Sep 14, 2012 at 6:54 PM, Erik Faye-Lund <kusmabite@xxxxxxxxx> wrote: >>> On Fri, Sep 14, 2012 at 6:41 PM, Erik Faye-Lund <kusmabite@xxxxxxxxx> 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 \ > && .... and in reality this echo use is not portable. http://ftp.gnu.org/old-gnu/Manuals/autoconf-2.53/html_node/Limitations-of-Builtins.html In popt 1_17 autogen.sh does red=; grn=; lgn=; blu=; std=; test "X$$TERM" != Xdumb \ && test -t 1 2>/dev/null \ && { \ red='^[[0;31m'; \ grn='^[[0;32m'; \ lgn='^[[1;32m'; \ blu='^[[1;34m'; \ std='^[[m'; \ } and Die() { color="$red" echo "${color}${_PROGNAME}: Error: $@${std}" >&2 exit 1 } Die "message here" > > or the inverse logic. This is what automake and popt autogen.sh does. > > Best Regards -- 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