From 576e8317ff9f9530a79e6254355538dbe24cc294 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Thu, 30 Sep 2021 20:33:47 +0200 Subject: [PATCH] Don't set CONFIGOPTS to empty string initially This results in passing empty argument to configure which, in turn, triggers "WARNING: you should use --build, --host, --target" from it as this argument is apparently misinterpreted as the host name. --- .github/workflows/ci.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index b9a5c4c81..cc9263912 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -53,7 +53,6 @@ jobs: SWIGLANG: ${{ matrix.SWIGLANG }} PY3: ${{ matrix.PY3 }} SWIG_FEATURES: ${{ matrix.SWIG_FEATURES }} - CONFIGOPTS: ${{ matrix.CONFIGOPTS }} CSTD: ${{ matrix.CSTD }} CPP11: ${{ matrix.CPP11 }} CPP14: ${{ matrix.CPP14 }} @@ -102,6 +101,9 @@ jobs: $CC --version $CXX --version + if test -n '${{ matrix.CONFIGOPTS }}'; then + CONFIGOPTS=${{ matrix.CONFIGOPTS }} + fi if test -n "$CPP11"; then CONFIGOPTS+=(--enable-cpp11-testing "CXXFLAGS=-std=c++11 $CXXFLAGS" "CFLAGS=-std=c11 $CFLAGS") && export CSTD=c11 && export CPPSTD=c++11; fi if test -n "$CPP14"; then CONFIGOPTS+=(--enable-cpp11-testing "CXXFLAGS=-std=c++14 $CXXFLAGS" "CFLAGS=-std=c11 $CFLAGS") && export CSTD=c11 && export CPPSTD=c++14; fi if test -n "$CPP17"; then CONFIGOPTS+=(--enable-cpp11-testing "CXXFLAGS=-std=c++17 $CXXFLAGS" "CFLAGS=-std=c17 $CFLAGS") && export CSTD=c17 && export CPPSTD=c++17; fi