From ddc7e032eaec5868124f9a952c7482849b362200 Mon Sep 17 00:00:00 2001
From: Ricardo Roldao <Ricardo.Roldao@partner.criticaltechworks.com>
Date: Fri, 10 May 2024 15:01:56 +0100
Subject: [PATCH] Build capicxx-someip-runtime on Windows

---
 .github/workflows/makefile.yml            | 75 ++++++++++++++++++-----
 CMakeLists.txt                            |  2 +-
 include/CommonAPI/SomeIP/OutputStream.hpp |  4 ++
 3 files changed, 65 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/makefile.yml b/.github/workflows/makefile.yml
index b50df53..a1c1b3b 100644
--- a/.github/workflows/makefile.yml
+++ b/.github/workflows/makefile.yml
@@ -7,7 +7,7 @@ on:
     branches: ["master"]
 
 jobs:
-  build:
+  build_on_ubuntu_22_04:
     runs-on: [ubuntu-22.04]
 
     steps:
@@ -26,20 +26,10 @@ jobs:
           repository: COVESA/vsomeip
           path: "vsomeip"
 
-      - name: "Install googletest"
+      - name: Install googletest
         run: |
-          sudo apt-get install libgtest-dev
-          cd /usr/src/gtest
-          sudo cmake CMakeLists.txt
-          sudo make
-          sudo cp lib/*.a /usr/lib
-          sudo ln -s /usr/lib/libgtest.a /usr/local/lib/libgtest.a
-          sudo ln -s /usr/lib/libgtest_main.a /usr/local/lib/libgtest_main.a
-
-      - name: "Build capicxx-core-runtime"
-        run: |
-          cmake -S capicxx-core-runtime -B build-core-runtime -D CMAKE_INSTALL_PREFIX=install
-          cmake --build build-core-runtime --target install
+          sudo apt-get update -qq
+          sudo apt-get install -y googletest
 
       - name: Ubuntu - Install boost 1.83.0 with gcc and x86
         uses: MarkusJx/install-boost@v2.4.4
@@ -52,6 +42,11 @@ jobs:
           arch: x86
           cache: true
 
+      - name: "Build capicxx-core-runtime"
+        run: |
+          cmake -S capicxx-core-runtime -B build-core-runtime -D CMAKE_INSTALL_PREFIX=install
+          cmake --build build-core-runtime --target install
+
       - name: "Build vsomeip"
         run: |
           cmake -S vsomeip -B vsomeip -D GTEST_ROOT=/usr/src/googletest -D BOOST_ROOT=/home/runner/boost/boost/ -D CMAKE_PREFIX_PATH=install -D CMAKE_INSTALL_PREFIX=install
@@ -62,4 +57,54 @@ jobs:
         run: |
           cmake -S . -B build-someip-runtime -D GTEST_ROOT=/usr/src/googletest -D CMAKE_PREFIX_PATH=install -D CMAKE_INSTALL_PREFIX=install
           cmake --build build-someip-runtime
-          cmake --install build-someip-runtime --strip
+
+  build_on_windows_latest:
+    runs-on: windows-latest
+
+    steps:
+      - name: "Checkout"
+        uses: actions/checkout@v3
+
+      - name: "Checkout capicxx-core-runtime"
+        uses: actions/checkout@v3
+        with:
+          repository: COVESA/capicxx-core-runtime
+          path: "capicxx-core-runtime"
+
+      - name: "Checkout vsomeip"
+        uses: actions/checkout@v3
+        with:
+          repository: COVESA/vsomeip
+          path: "vsomeip"
+
+      - name: "Checkout googletest"
+        uses: actions/checkout@v3
+        with:
+          repository: google/googletest
+          path: "googletest"
+
+      - name: Windows - Install boost 1.83.0 with gcc and x86
+        uses: MarkusJx/install-boost@v2.4.4
+        id: windows-gcc-1_83-x86
+        with:
+          boost_version: 1.83.0
+          platform: windows
+          boost_install_dir: C:\runner
+          toolset: msvc
+          arch: x86
+          cache: true
+
+      - name: "Build capicxx-core-runtime"
+        run: |
+          cmake -S capicxx-core-runtime -B build-core-runtime -D BOOST_ROOT=C:\runner\boost -DCMAKE_INSTALL_PREFIX=${{ runner.workspace }}\install
+          cmake --build build-core-runtime --target install
+
+      - name: "Build vsomeip"
+        run: |
+          cmake -S vsomeip -B build-vsomeip -D GTEST_ROOT=${{ runner.workspace }}\capicxx-someip-runtime\googletest -D BOOST_ROOT=C:\runner\boost -D CMAKE_INSTALL_PREFIX=${{ runner.workspace }}\install
+          cmake --build build-vsomeip --target install
+
+      - name: "Build capicxx-someip-runtime"
+        run: |
+          cmake -B build-someip-runtime -D GTEST_ROOT=${{ runner.workspace }}\capicxx-someip-runtime\googletest -D BOOST_ROOT=C:\runner\boost -D CMAKE_PREFIX_PATH=${{ runner.workspace }}\install .
+          cmake --build build-someip-runtime
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 22fa95e..13cdf49 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -63,7 +63,7 @@ else()
   MESSAGE( STATUS "Boost was not found!")
 endif()
 include_directories( ${Boost_INCLUDE_DIR} )
-set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_CRT_SECURE_NO_WARNINGS /wd4503")
+set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /std:c++17 -D_CRT_SECURE_NO_WARNINGS /wd4503")
 link_directories(${Boost_LIBRARY_DIR})
 else()
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Wextra -Wformat -Wformat-security -Wconversion -fexceptions -fstrict-aliasing -fstack-protector -fasynchronous-unwind-tables -fno-omit-frame-pointer -DCOMMONAPI_INTERNAL_COMPILATION -D_GLIBCXX_USE_NANOSLEEP -DBOOST_LOG_DYN_LINK -pthread -fvisibility=hidden")
diff --git a/include/CommonAPI/SomeIP/OutputStream.hpp b/include/CommonAPI/SomeIP/OutputStream.hpp
index 54b5542..8e965d6 100644
--- a/include/CommonAPI/SomeIP/OutputStream.hpp
+++ b/include/CommonAPI/SomeIP/OutputStream.hpp
@@ -568,7 +568,11 @@ class OutputStream: public CommonAPI::OutputStream<OutputStream> {
                 byte_t raw[sizeof(Type_)];
             } value;
             value.typed = _value;
+#ifndef _WIN32
             if ((__BYTE_ORDER == __LITTLE_ENDIAN) != isLittleEndian_) {
+#else
+            if (!isLittleEndian_) {
+#endif
                 byte_t reordered[sizeof(Type_)];
                 byte_t *source = &value.raw[sizeof(Type_) - 1];
                 byte_t *target = reordered;