[Libreoffice-commits] core.git: configure.ac

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Thu Jul 26 09:27:32 UTC 2018


 configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 094f9f48b227e9e9cb7218555cfbd8868d9646d7
Author:     Luke Deller <luke at deller.id.au>
AuthorDate: Thu Jul 5 22:52:49 2018 +1000
Commit:     Michael Stahl <Michael.Stahl at cib.de>
CommitDate: Thu Jul 26 11:27:06 2018 +0200

    Avoid configure error when warning with no TERM
    
    When configure wants to show a warning, it tries to check whether the
    terminal supports colour.  This check produces an error message
    when running with no TERM environment variable set (seen on a tinderbox)
    
    Fix this check so that it tolerates failure of the `tput colors` command
    
    Change-Id: I263136142120cf66fe900df0deb74d2fa6e5d1d0
    Reviewed-on: https://gerrit.libreoffice.org/57009
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>

diff --git a/configure.ac b/configure.ac
index 9b31d8904b54..c02a29120dff 100644
--- a/configure.ac
+++ b/configure.ac
@@ -112,7 +112,7 @@ add_warning()
     if test "$have_WARNINGS" = "no"; then
         echo "*************************************" > warn
         have_WARNINGS="yes"
-        if which tput >/dev/null 2>/dev/null && test `tput colors` -ge 8; then
+        if which tput >/dev/null && test "`tput colors 2>/dev/null || echo 0`" -ge 8; then
             dnl <esc> as actual byte (U+1b), [ escaped using quadrigraph @<:@
             COLORWARN='*@<:@1;33;40m WARNING @<:@0m:'
         else


More information about the Libreoffice-commits mailing list