On 5/3/19 5:35 PM, Frederic Lecaille wrote:
On 5/3/19 3:44 PM, Илья Шипицин wrote:

пт, 3 мая 2019 г. в 18:42, Tim Düsterhus <t...@bastelstu.be <mailto:t...@bastelstu.be>>:


    Am 03.05.19 um 15:39 schrieb Илья Шипицин:
     > when I played with enabling travis-ci, I tried to set TMPDIR
     > however I was not lucky enough.
     > Later Tim added "sed" magic to .travis.yml
     > personally, I do not understand why "sed" is better than
    assigning TMPDIR
     > directly.

    I did not try using TMPDIR=/tmp or something like that, because I
    thought there must be a reason why it's that strange long path.

I tried /tmp and /var/tmp
it seems that not any filesystem on osx can hold network socket (at least from my point of view)

try to create a working directory owned by the user which run the reg test :

    $ mkdir -p ~/tmp/
    $ TMPDIR=~/tmp make reg-tests

I confirm that with such a value everything work on all OS'es (https://travis-ci.com/haproxyFred/haproxy)

The attached patch should fix this issue.

Thank you Tim, Ilya.

>From fc9decae9ec679038dc494ad612dd3eb144de408 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20L=C3=A9caille?= <flecai...@haproxy.com>
Date: Fri, 3 May 2019 19:16:02 +0200
Subject: [PATCH] BUILD: travis: TMPDIR replacement.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

TMPDIR default value is too long especially on OSX systems.
We decided to shorten it for all the OS'es.

Thank you to Tim Düsterhus and Ilya for having helped on this issue.
 .travis.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index f689fe982..7475ad028 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -31,12 +31,12 @@ before_script:
   # This is a fix for the super long TMPDIR on Mac making
   # the unix socket path names exceed the maximum allowed
   # length.
-  - sed -i'.original' '/TESTDIR=.*haregtests/s/haregtests-.*XXXXXX/regtest.XXX/' scripts/run-regtests.sh
+  - mkdir ~/tmp
   - ./haproxy -vv
-  - env VTEST_PROGRAM=../vtest/vtest make reg-tests
+  - env TMPDIR=~/tmp VTEST_PROGRAM=../vtest/vtest make reg-tests
   - |

Reply via email to