Malvarado/windows ci (#170)

* Have CI build under windows * Fix syntax error * Source updates
carnegierobotics · Nov 25, 2024 · 87aa331 · 87aa331
1 parent e9d8bdc
commit 87aa331
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 5 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -32,3 +32,18 @@ jobs:
           cmake --build build -j "$(nproc)"
           make -C build test
           cmake --install build
+
+  buildwindows:
+    name: build for windows
+    runs-on: windows-2022
+
+    steps:
+    - uses: actions/checkout@v4
+    - id: build-release
+      run: |
+          cmake -B ${{github.workspace}}/build -DCMAKE_BUILD_TYPE=Release
+          cmake --build ${{github.workspace}}/build --config Release --target install -- /m:10
+    - id: build-debug
+      run: |
+          cmake -B ${{github.workspace}}/build -DCMAKE_BUILD_TYPE=Debug
+          cmake --build ${{github.workspace}}/build --config Debug --target install -- /m:10
diff --git a/source/LibMultiSense/details/public.cc b/source/LibMultiSense/details/public.cc
@@ -1345,15 +1345,15 @@ Status impl::getMtu(int32_t& mtu)
 
 Status impl::setBestMtu()
 {
-    uint cur_mtu = MAX_MTU_SIZE;
-    uint max_mtu = MAX_MTU_SIZE;
-    uint min_mtu = MIN_MTU_SIZE;
-    uint bisections = 0;
+    uint32_t cur_mtu = MAX_MTU_SIZE;
+    uint32_t max_mtu = MAX_MTU_SIZE;
+    uint32_t min_mtu = MIN_MTU_SIZE;
+    uint32_t bisections = 0;
     Status status = Status_Ok;
 
     //
     // v2.2 and older do not support testing MTU
-    
+
     if (m_sensorVersion.firmwareVersion <= 0x0202)
         return Status_Unsupported;