CI: Fix freebsd python2 / bash tests

.github/workflows/ci.yml

@@ -60,17 +60,16 @@ jobs:
 
 
   build-freebsd:
-    runs-on: macos-12
+    runs-on: ubuntu-22.04
     name: Build and test FreeBSD executable
     steps:
     - uses: actions/checkout@v3
     - name: FreeBSD
       id: test
-      uses: vmactions/freebsd-vm@v0
+      uses: vmactions/freebsd-vm@v1
       with:
         usesh: true
-        sync: rsync
-        prepare: pkg install -y binutils cmake elfutils python bash git
+        prepare: pkg install -y binutils cmake elfutils python bash git python2
         run: |
           chmod u+x .github/workflows/freebsd-build.sh .github/workflows/ci-run-tests.sh
           .github/workflows/freebsd-build.sh x86_64

.github/workflows/freebsd-build.sh

@@ -14,6 +14,10 @@ run () {
   cmake -DCMAKE_INSTALL_PREFIX=/usr/local ../tests || exit 64
   make || exit 64
   cd ..
+
+  chmod u+x .github/workflows/test-executable.sh
+  kcov --include-pattern=test-executable.sh coverage .github/workflows/test-executable.sh
+  cat coverage/test-executable.sh/coverage.json
 }
 
 run "$@"

src/configuration.cc

@@ -5,6 +5,7 @@
 #include <getopt.h>
 #include <limits.h>
 #include <sys/stat.h>
+#include <sys/param.h>
 #include <sys/types.h>
 #include <unistd.h>
 #include <map>

src/engines/bash-engine.cc

@@ -153,6 +153,8 @@ class BashEngine : public ScriptEngineBase
 			unsigned int argc = conf.getArgc();
 			int xtraceFd = 782; // Typical bash users use 3,4 etc but not high fd numbers (?)
 			{
+// This doesn't work on (recent?) FreeBSDs, so disable for now
+#ifndef __FreeBSD__
 				// However, the range of fd numbers that this process can use is limited by the caller of this process.
 				// The following code specification will require discussion with many kcov users.
 				struct rlimit rlim;
@@ -161,6 +163,7 @@ class BashEngine : public ScriptEngineBase
 					return false;
 				}
 				xtraceFd = rlim.rlim_cur / 4;
+#endif
 			}
 			const std::string command = conf.keyAsString("bash-command");
 			bool usePS4 = conf.keyAsInt("bash-use-ps4");

tests/CMakeLists.txt

@@ -143,7 +143,7 @@ add_executable(multi_1 merge-tests/file.c merge-tests/main_1.c)
 add_executable(multi_2 merge-tests/file.c merge-tests/main_2.c)
 
 add_executable(setpgid-kill setpgid-kill/setpgid-kill-main.cc ../src/utils.cc)
-target_link_libraries(setpgid-kill ${ZLIB_LIBRARIES} ${CURL_LIBRARIES})
+target_link_libraries(setpgid-kill ${ZLIB_LIBRARIES} CURL::libcurl)
 
 add_executable(issue31 daemon/test-issue31.cc)
 target_link_libraries(issue31 ${CMAKE_THREAD_LIBS_INIT})