Merge pull request #904 from bsbernd/s-bit-fusermount

Add back s-bit for compiled fusermount

4 files changed, 82 insertions, 42 deletions

diff --git a/test/ci-build.sh b/test/ci-build.sh
index a023c15..0307865 100755
--- a/test/ci-build.sh
+++ b/test/ci-build.sh
@@ -1,8 +1,8 @@
-#!/bin/bash
+#!/bin/bash -x
 
 set -e
 
-TEST_CMD="python3 -m pytest --maxfail=99 test/"
+TEST_CMD="pytest -v --maxfail=1 --log-level=DEBUG --log-cli-level=DEBUG test/"
 SAN="-Db_sanitize=address,undefined"
 
 # not default
@@ -30,38 +30,48 @@ export LSAN_OPTIONS="suppressions=$(pwd)/lsan_suppress.txt"
 export ASAN_OPTIONS="detect_leaks=1"
 export CC
 
-# Standard build
-for CC in gcc gcc-9 gcc-10 clang; do
-    echo "=== Building with ${CC} ==="
-    mkdir build-${CC}; pushd build-${CC}
-    if [ "${CC}" == "clang" ]; then
-        export CXX="clang++"
-        export TEST_WITH_VALGRIND=false
-    else
-        export TEST_WITH_VALGRIND=true
-    fi
-    if [ ${CC} == 'gcc-7' ]; then
-        build_opts='-D b_lundef=false'
-    else
-        build_opts=''
-    fi
-    if [ ${CC} == 'gcc-10' ]; then
-        build_opts='-Dc_args=-flto=auto'
-    else
-        build_opts=''
-    fi
-    meson setup -Dprefix=${PREFIX_DIR} -D werror=true ${build_opts} "${SOURCE_DIR}" || (cat meson-logs/meson-log.txt; false)
-    ninja
-    sudo ninja install
-
-    # libfuse will first try the install path and then system defaults
-    sudo chmod 4755 ${PREFIX_DIR}/bin/fusermount3
-
-    ${TEST_CMD}
-    popd
-    rm -fr build-${CC}
-    sudo rm -fr ${PREFIX_DIR}
-done
+non_sanitized_build()
+(
+    echo "Standard build (without sanitizers)"
+    for CC in gcc gcc-9 gcc-10 clang; do
+        echo "=== Building with ${CC} ==="
+        mkdir build-${CC}; pushd build-${CC}
+        if [ "${CC}" == "clang" ]; then
+            export CXX="clang++"
+            export TEST_WITH_VALGRIND=false
+        else
+            unset CXX
+            export TEST_WITH_VALGRIND=true
+        fi
+        if [ ${CC} == 'gcc-7' ]; then
+            build_opts='-D b_lundef=false'
+        else
+            build_opts=''
+        fi
+        if [ ${CC} == 'gcc-10' ]; then
+            build_opts='-Dc_args=-flto=auto'
+        else
+            build_opts=''
+        fi
+
+        meson setup -Dprefix=${PREFIX_DIR} -D werror=true ${build_opts} "${SOURCE_DIR}" || (cat meson-logs/meson-log.txt; false)
+        ninja
+        sudo ninja install
+
+        # libfuse will first try the install path and then system defaults
+        sudo chmod 4755 ${PREFIX_DIR}/bin/fusermount3
+
+        # also needed for some of the tests
+        sudo chown root:root util/fusermount3
+        sudo chmod 4755 util/fusermount3
+
+        ${TEST_CMD}
+        popd
+        rm -fr build-${CC}
+        sudo rm -fr ${PREFIX_DIR}
+
+    done
+)
 
 sanitized_build()
 (
@@ -78,11 +88,24 @@ sanitized_build()
     # bug, cf. https://groups.google.com/forum/#!topic/mesonbuild/tgEdAXIIdC4
     meson configure -D b_lundef=false
 
-    meson configure
+    # additional options
+    if [ -n "$@" ]; then
+        meson configure "$@"
+    fi
+
+    # print all options
+    meson configure --no-pager
+
+    # reconfigure to ensure it uses all additional options
+    meson setup --reconfigure "${SOURCE_DIR}"
     ninja
     sudo ninja install
     sudo chmod 4755 ${PREFIX_DIR}/bin/fusermount3
 
+    # also needed for some of the tests
+    sudo chown root:root util/fusermount3
+    sudo chmod 4755 util/fusermount3
+
     # Test as root and regular user
     sudo ${TEST_CMD}
     # Cleanup temporary files (since they are now owned by root)
@@ -96,15 +119,17 @@ sanitized_build()
 )
 
 # Sanitized build
-CC=clang
-CXX=clang++
+export CC=clang
+export CXX=clang++
 TEST_WITH_VALGRIND=false
-sanitized_build $SAN
+sanitized_build
 
 # Sanitized build without libc versioned symbols
-CC=clang
-CXX=clang++
-sanitized_build
+export CC=clang
+export CXX=clang++
+sanitized_build "-Ddisable-libc-symbol-version=true"
+
+non_sanitized_build
 
 # Documentation.
 (cd "${SOURCE_DIR}"; doxygen doc/Doxyfile)
diff --git a/test/pytest.ini b/test/pytest.ini
index 036a9f4..bbc8de8 100644
--- a/test/pytest.ini
+++ b/test/pytest.ini
@@ -2,3 +2,5 @@
 addopts = --verbose --assert=rewrite --tb=native -x -r a
 markers = 
     uses_fuse: Indicates that FUSE is supported.
+log_cli=true
+faulthandler_timeout=60
diff --git a/test/test_ctests.py b/test/test_ctests.py
index 951a34b..55db156 100644
--- a/test/test_ctests.py
+++ b/test/test_ctests.py
@@ -9,10 +9,11 @@ import subprocess
 import pytest
 import platform
 import sys
+import os
 from looseversion import LooseVersion
 from util import (wait_for_mount, umount, cleanup, base_cmdline,
                   safe_sleep, basename, fuse_test_marker, fuse_caps,
-                  fuse_proto)
+                  fuse_proto, create_tmpdir)
 from os.path import join as pjoin
 import os.path
 
@@ -29,6 +30,9 @@ def test_write_cache(tmpdir, writeback, output_checker):
     # deadlock in valgrind, it probably assumes that until close() returns,
     # control does not come to the program.
     mnt_dir = str(tmpdir)
+    print("mnt_dir: '" + mnt_dir + "'")
+    create_tmpdir(mnt_dir)
+
     cmdline = [ pjoin(basename, 'test', 'test_write_cache'),
                 mnt_dir ]
     if writeback:
@@ -50,6 +54,7 @@ if fuse_proto >= (7,15):
 @pytest.mark.parametrize("notify", (True, False))
 def test_notify1(tmpdir, name, notify, output_checker):
     mnt_dir = str(tmpdir)
+    create_tmpdir(mnt_dir)
     cmdline = base_cmdline + \
               [ pjoin(basename, 'example', name),
                 '-f', '--update-interval=1', mnt_dir ]
@@ -70,6 +75,7 @@ def test_notify1(tmpdir, name, notify, output_checker):
         else:
             assert read1 == read2
     except:
+        print("Failure in notify test: '" + str(cmdline) + "'")
         cleanup(mount_process, mnt_dir)
         raise
     else:
@@ -80,6 +86,7 @@ def test_notify1(tmpdir, name, notify, output_checker):
 @pytest.mark.parametrize("notify", (True, False))
 def test_notify_file_size(tmpdir, notify, output_checker):
     mnt_dir = str(tmpdir)
+    create_tmpdir(mnt_dir)
     cmdline = base_cmdline + \
               [ pjoin(basename, 'example', 'invalidate_path'),
                 '-f', '--update-interval=1', mnt_dir ]
diff --git a/test/util.py b/test/util.py
index 0a1fa93..623b031 100644
--- a/test/util.py
+++ b/test/util.py
@@ -151,6 +151,12 @@ def powerset(iterable):
   return itertools.chain.from_iterable(
       itertools.combinations(s, r) for r in range(len(s)+1))
 
+def create_tmpdir(mnt_dir):
+    if not os.path.exists(mnt_dir):
+        print("makedirs: '" + mnt_dir + "'")
+        os.makedirs(mnt_dir)
+    else:
+        print("mnt_dir exists: '" + mnt_dir + "'")
 
 # Use valgrind if requested
 if os.environ.get('TEST_WITH_VALGRIND', 'no').lower().strip() \