The openDAQ ci workflow build MacOS refactoring

.github/actions/build-sdk/action.yml

@@ -0,0 +1,200 @@
+name: Build openDAQ SDK
+description: Configure, build and test openDAQ SDK
+
+inputs:
+  additional-dependencies:
+    description: 'Additional packages to install (apt on Linux, choco on Windows)'
+    required: false
+    default: ''
+  cmake-generator:
+    description: 'CMake generator (e.g., "Ninja", "Visual Studio 17 2022")'
+    required: false
+    default: ''
+  cmake-config-preset:
+    description: 'CMake configure preset name'
+    required: false
+    default: ''
+  cmake-config-args:
+    description: 'Additional CMake configure arguments (e.g., -D flags)'
+    required: false
+    default: ''
+  cmake-build-type:
+    description: 'Build configuration (Release, Debug). Required for multi-config generators (Visual Studio)'
+    required: false
+    default: ''
+  enable-32bit:
+    description: 'Build for 32-bit architecture'
+    required: false
+    default: 'false'
+  enable-tests:
+    description: 'Run tests after build'
+    required: false
+    default: 'false'
+  ctest-preset:
+    description: 'CTest preset name'
+    required: false
+    default: ''
+
+outputs:
+  gtest-xml-path:
+    description: 'Path to GTest XML reports directory'
+    value: ${{ steps.test.outputs.gtest-xml-path }}
+
+runs:
+  using: composite
+  steps:
+    - name: Check inputs
+      shell: bash
+      env:
+        CMAKE_GENERATOR: ${{ inputs.cmake-generator }}
+      run: |
+        if [ -z "$CMAKE_GENERATOR" ]; then
+          echo "::error::✗ The cmake-generator input is not set."
+          exit 1
+        fi
+
+    - name: Set up Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.14'
+        architecture: ${{ inputs.enable-32bit == 'true' && 'x86' || 'x64' }}
+
+    - name: Setup cmake
+      uses: jwlawson/actions-setup-cmake@v2
+
+    - name: Install Windows dependencies
+      if: runner.os == 'Windows' && inputs.additional-dependencies != ''
+      shell: pwsh
+      run: choco install ${{ inputs.additional-dependencies }} -y --no-progress
+
+    - name: Install Linux dependencies
+      if: runner.os == 'Linux'
+      shell: bash
+      run: |
+        sudo apt-get update && sudo apt-get install -y --no-install-recommends \
+          ${{ inputs.additional-dependencies || '' }} \
+          lld mono-runtime libmono-system-json-microsoft4.0-cil libmono-system-data4.0-cil \
+          libx11-dev libxi-dev libxcursor-dev libxrandr-dev libgl-dev libudev-dev libfreetype6-dev
+
+    - name: Install macOS dependencies
+      if: runner.os == 'macOS'
+      shell: bash
+      run: |
+        HOMEBREW_NO_AUTO_UPDATE=1 brew install mono ${{ inputs.additional-dependencies }}
+
+    - name: Install ninja-build
+      if: inputs.cmake-generator == 'Ninja'
+      uses: seanmiddleditch/gha-setup-ninja@v5
+
+    - name: Setup MSYS2 MINGW64
+      if: runner.os == 'Windows'
+      id: msys2
+      uses: msys2/setup-msys2@v2
+      with:
+        msystem: MINGW64
+        update: false
+        install: mingw-w64-x86_64-crt-git mingw-w64-x86_64-gcc
+
+    - name: Add MSYS2 MINGW64 to PATH
+      if: runner.os == 'Windows'
+      shell: bash
+      run: |
+        echo "${{ steps.msys2.outputs.msys2-location }}/mingw64/bin" >> $GITHUB_PATH
+        echo "Added MSYS2 MINGW64 to PATH: ${{ steps.msys2.outputs.msys2-location }}/mingw64/bin"
+
+    # https://github.com/actions/runner-images/issues/10001
+    - name: Setup MSVC toolchain
+      if: runner.os == 'Windows'
+      uses: ilammy/msvc-dev-cmd@v1
+
+    - name: Install Python dependencies
+      shell: bash
+      run: |
+        echo "Installing numpy for Python versions: $(python --version)"
+        python -m pip install numpy
+        echo "Done installing Python dependencies"
+
+    - name: Configure
+      id: cmake-config
+      shell: bash
+      env:
+        CMAKE_BUILD_TYPE: ${{ inputs.cmake-build-type }}
+        CMAKE_GENERATOR: ${{ inputs.cmake-generator }}
+        CI_GIT_BRANCH: ${{ github.head_ref }}
+      run: |
+        CMAKE_BUILD_PATH="build"
+        mkdir -p "$CMAKE_BUILD_PATH"
+        CMAKE_BUILD_PATH=$(cd "$CMAKE_BUILD_PATH" && pwd)
+        echo "build-path=$CMAKE_BUILD_PATH" >> "$GITHUB_OUTPUT"
+
+        CMAKE_ARGS=""
+        if [ -n "${{ inputs.cmake-config-preset }}" ]; then
+          CMAKE_ARGS="--preset ${{ inputs.cmake-config-preset }}"
+        fi
+
+        # Visual Studio specific options
+        if [[ "$CMAKE_GENERATOR" == "Visual Studio"* ]]; then
+          CMAKE_ARGS="$CMAKE_ARGS -DOPENDAQ_MSVC_SINGLE_PROCESS_BUILD=ON"
+        fi
+
+        if [ -n "$CMAKE_BUILD_TYPE" ]; then
+          CMAKE_ARGS="$CMAKE_ARGS -DCMAKE_BUILD_TYPE=$CMAKE_BUILD_TYPE"
+        fi
+
+        CMAKE_ARGS="$CMAKE_ARGS -DCI_GIT_BRANCH=$CI_GIT_BRANCH"
+
+        echo "::group::CMake Configure"
+        cmake -B "$CMAKE_BUILD_PATH" -G "$CMAKE_GENERATOR" $CMAKE_ARGS ${{ inputs.cmake-config-args }}
+        echo "::endgroup::"
+
+    - name: Build
+      shell: bash
+      env:
+        SHORT_SHA: ci # .NET bindings version suffix (e.g., 1.0.0-beta-ci)
+        CMAKE_BUILD_TYPE: ${{ inputs.cmake-build-type }}
+        CMAKE_BUILD_PATH: ${{ steps.cmake-config.outputs.build-path }}
+      run: |
+        CMAKE_BUILD_ARGS=""
+        if [ -n "$CMAKE_BUILD_TYPE" ]; then
+          CMAKE_BUILD_ARGS="--config $CMAKE_BUILD_TYPE"
+        fi
+
+        echo "::group::CMake Build"
+        cmake --build "$CMAKE_BUILD_PATH" $CMAKE_BUILD_ARGS
+        echo "::endgroup::"
+
+    - name: Test
+      id: test
+      if: inputs.enable-tests == 'true'
+      shell: bash
+      env:
+        CMAKE_BUILD_TYPE: ${{ inputs.cmake-build-type }}
+        CMAKE_BUILD_PATH: ${{ steps.cmake-config.outputs.build-path }}
+      run: |
+        GTEST_XML_PATH="$CMAKE_BUILD_PATH/test_results"
+        mkdir -p "$GTEST_XML_PATH"
+
+        if command -v cygpath &> /dev/null; then
+          GTEST_XML_PATH=$(cygpath -w "$GTEST_XML_PATH")
+        fi
+
+        export GTEST_OUTPUT="xml:$GTEST_XML_PATH/"
+        echo "gtest-xml-path=$GTEST_XML_PATH" >> "$GITHUB_OUTPUT"
+
+        CTEST_ARGS=""
+        if [ -n "${{ inputs.ctest-preset }}" ]; then
+          CTEST_ARGS="--preset ${{ inputs.ctest-preset }}"
+        fi
+
+        if [ -n "$CMAKE_BUILD_TYPE" ]; then
+          CTEST_ARGS="$CTEST_ARGS --build-config $CMAKE_BUILD_TYPE"
+        fi
+
+        SUDO_CMD=""
+        if command -v sudo &> /dev/null; then
+          SUDO_CMD="sudo -E"
+        fi
+
+        echo "::group::CTest"
+        $SUDO_CMD ctest --test-dir "$CMAKE_BUILD_PATH" --output-on-failure $CTEST_ARGS
+        echo "::endgroup::"

.github/patches/diagnose-skip-samples-ub.patch

@@ -0,0 +1,54 @@
+diff --git a/core/opendaq/reader/tests/test_stream_reader.cpp b/core/opendaq/reader/tests/test_stream_reader.cpp
+index 50c68c0b..b912eea7 100644
+--- a/core/opendaq/reader/tests/test_stream_reader.cpp
++++ b/core/opendaq/reader/tests/test_stream_reader.cpp
+@@ -1748,16 +1748,20 @@ TYPED_TEST(StreamReaderTest, SkipSamplesOnePacket)
+     {
+         for (size_t i = 0; i < 10; ++i)
+         {
+-            ASSERT_EQ(firstBatchSamples[i], TypeParam(typename TypeParam::Type(i * 11.1)));
+-            ASSERT_EQ(secondBatchSamples[i], TypeParam(typename TypeParam::Type((40 + i) * 11.1)));
++            TypeParam expectedFirst = TypeParam(typename TypeParam::Type(i * 11.1));
++            TypeParam expectedSecond = TypeParam(typename TypeParam::Type((40 + i) * 11.1));
++            ASSERT_EQ(firstBatchSamples[i], expectedFirst);
++            ASSERT_EQ(secondBatchSamples[i], expectedSecond);
+         }
+     }
+     else
+     {
+         for (size_t i = 0; i < 10; ++i)
+         {
+-            ASSERT_EQ(firstBatchSamples[i], (TypeParam)(i * 11.1));
+-            ASSERT_EQ(secondBatchSamples[i], (TypeParam) ((40 + i) * 11.1));
++            TypeParam expectedFirst = (TypeParam)(i * 11.1);
++            TypeParam expectedSecond = (TypeParam)((40 + i) * 11.1);
++            ASSERT_EQ(firstBatchSamples[i], expectedFirst);
++            ASSERT_EQ(secondBatchSamples[i], expectedSecond);
+         }
+     }
+ }
+@@ -1819,16 +1823,20 @@ TYPED_TEST(StreamReaderTest, SkipSamplesBetweenPackets)
+     {
+         for (size_t i = 0; i < 10; ++i)
+         {
+-            ASSERT_EQ(firstBatchSamples[i], TypeParam(typename TypeParam::Type(i * 11.1)));
+-            ASSERT_EQ(secondBatchSamples[i], TypeParam(typename TypeParam::Type((40 + i) * 11.1)));
++            TypeParam expectedFirst = TypeParam(typename TypeParam::Type(i * 11.1));
++            TypeParam expectedSecond = TypeParam(typename TypeParam::Type((40 + i) * 11.1));
++            ASSERT_EQ(firstBatchSamples[i], expectedFirst);
++            ASSERT_EQ(secondBatchSamples[i], expectedSecond);
+         }
+     }
+     else
+     {
+         for (size_t i = 0; i < 10; ++i)
+         {
+-            ASSERT_EQ(firstBatchSamples[i], (TypeParam) (i * 11.1));
+-            ASSERT_EQ(secondBatchSamples[i], (TypeParam) ((40 + i) * 11.1));
++            TypeParam expectedFirst = (TypeParam)(i * 11.1);
++            TypeParam expectedSecond = (TypeParam)((40 + i) * 11.1);
++            ASSERT_EQ(firstBatchSamples[i], expectedFirst);
++            ASSERT_EQ(secondBatchSamples[i], expectedSecond);
+         }
+     }
+ }