Make CI workflow.

Author: sobomax

.github/workflows/main.yml

@@ -0,0 +1,76 @@
+name: CI
+
+on:
+  push:
+  pull_request:
+  workflow_dispatch:
+
+jobs:
+  posix:
+    name: ${{ matrix.name }}
+    runs-on: ${{ matrix.os }}
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - name: Ubuntu 22.04 GCC
+            os: ubuntu-22.04
+            cc: gcc
+            cxx: g++
+            packages: build-essential
+            cc_flags: -march=native
+
+          - name: Ubuntu 22.04 Clang
+            os: ubuntu-22.04
+            cc: clang
+            cxx: clang++
+            packages: clang
+            cc_flags: -march=native
+
+          - name: Ubuntu 24.04 GCC
+            os: ubuntu-24.04
+            cc: gcc
+            cxx: g++
+            packages: build-essential
+            cc_flags: -march=native
+
+          - name: Ubuntu 24.04 Clang
+            os: ubuntu-24.04
+            cc: clang
+            cxx: clang++
+            packages: clang
+            cc_flags: -march=native
+
+    env:
+      CC: ${{ matrix.cc }}
+      CXX: ${{ matrix.cxx }}
+      CC_FLAGS: ${{ matrix.cc_flags }}
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Install dependencies
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y cmake expect netcat-openbsd ${{ matrix.packages }}
+
+      - name: Show tool versions
+        run: |
+          cmake --version
+          "${CC}" --version
+
+      - name: Configure
+        run: |
+          cmake \
+            -DCMAKE_C_FLAGS="-Werror ${CC_FLAGS}" \
+            -S platforms/posix \
+            -B platforms/posix/build
+
+      - name: Build
+        run: cmake --build platforms/posix/build
+
+      - name: Test
+        working-directory: platforms/posix/build
+        run: ctest -V

platforms/posix/CMakeLists.txt

@@ -93,7 +93,10 @@ ENDIF (IS_BIG_ENDIAN)
 
 enable_testing()
 
-add_test(basic ${CMAKE_SOURCE_DIR}/../../scripts/do-test.sh ${CMAKE_SOURCE_DIR})
+add_test(NAME basic COMMAND
+  ${CMAKE_COMMAND} -E env BUILDDIR=${CMAKE_CURRENT_BINARY_DIR}
+  ${CMAKE_SOURCE_DIR}/../../scripts/do-test.sh ${CMAKE_SOURCE_DIR}
+)
 add_test(NAME usipy_test_uac COMMAND usipy_test_uac)
 add_test(NAME usipy_test_ua COMMAND usipy_test_ua)
 

platforms/posix/scripts/test_start.sub

@@ -1 +1,8 @@
-${BUILDDIR}/usipy_test 2>&1 | tee "${MLOG}" &
+MON_PIDFILE="${BUILDDIR}/usipy_test.pid"
+rm -f "${MON_PIDFILE}"
+(
+  "${BUILDDIR}/usipy_test" 2>&1 &
+  echo ${!} > "${MON_PIDFILE}"
+  wait ${!}
+) | tee "${MLOG}" &
+MON_PIPE_PID=${!}

platforms/posix/usipy_test_ua.c

@@ -43,7 +43,10 @@ bind_loopback_udp(void)
     sin.sin_family = AF_INET;
     sin.sin_addr.s_addr = htonl(INADDR_LOOPBACK);
     sin.sin_port = 0;
-    assert(bind(sock, (struct sockaddr *)&sin, sizeof(sin)) == 0);
+    if (bind(sock, (struct sockaddr *)&sin, sizeof(sin)) != 0) {
+        perror("bind_loopback_udp bind");
+        assert(0);
+    }
     return (sock);
 }
 

platforms/posix/usipy_test_uac.c

@@ -318,7 +318,10 @@ bind_loopback_udp(void)
     sin.sin_family = AF_INET;
     sin.sin_port = htons(0);
     sin.sin_addr.s_addr = htonl(INADDR_LOOPBACK);
-    assert(bind(sock, (const struct sockaddr *)&sin, sizeof(sin)) == 0);
+    if (bind(sock, (const struct sockaddr *)&sin, sizeof(sin)) != 0) {
+        perror("bind_loopback_udp bind");
+        assert(0);
+    }
     return (sock);
 }
 

scripts/do-test.sh

@@ -10,6 +10,22 @@ TOOLSDIR=`dirname ${TOOLSPATH}`
 
 MLOG="${BUILDDIR}/monitor.log"
 TESTS="empty foobar 100trying ACK OPTIONS INVITE CANCEL 200OK"
+MON_PID=
+MON_PIPE_PID=
+
+cleanup()
+{
+  if [ ! -z "${MON_PID}" ]
+  then
+    kill -TERM "${MON_PID}" 2>/dev/null || true
+    wait "${MON_PID}" 2>/dev/null || true
+  fi
+  if [ ! -z "${MON_PIPE_PID}" ]
+  then
+    wait "${MON_PIPE_PID}" 2>/dev/null || true
+  fi
+}
+trap cleanup EXIT INT TERM
 
 if [ -e "${MLOG}" ]
 then
@@ -18,7 +34,17 @@ fi
 
 . "${SRCDIR}/scripts/test_start.sub"
 MON_RC=${?}
-MON_PID=${!}
+i=0
+while [ ${i} -lt 10 ] && [ ! -s "${MON_PIDFILE}" ]
+do
+  sleep 1
+  i=$((${i} + 1))
+done
+if [ ! -s "${MON_PIDFILE}" ]
+then
+  exit 1
+fi
+MON_PID=`cat "${MON_PIDFILE}"`
 
 . "${SRCDIR}/scripts/test_waitready.sub"
 
@@ -36,8 +62,8 @@ then
   done
   sleep 3
 fi
-kill -TERM ${MON_PID}
-wait ${MON_PID} || true
+cleanup
+MON_PID=
 if [ -z "${BRD_IP}" ]
 then
   exit 1

scripts/test.common.sub

@@ -17,5 +17,5 @@ then
   . "${SRCDIR}/scripts/test.common.sub"
 fi
 
-BUILDDIR="${SRCDIR}/build"
+BUILDDIR="${BUILDDIR:-${SRCDIR}/build}"
 DIFF="diff -u"