Merge pull request #218 from robinlinden/clang-ci

Add Clang CI jobs

Author: oberstet

.github/workflows/ci-linux.yaml

@@ -27,6 +27,19 @@ jobs:
             conan: true
             cmake-args: -DCMAKE_PREFIX_PATH=`pwd`/build -DCMAKE_MODULE_PATH=`pwd`/build
 
+          - name: ubuntu-20.04-clang-10
+            os: ubuntu-20.04
+            compiler: clang
+            version: "10"
+            cmake-args: -DAUTOBAHN_USE_LIBCXX=NO
+
+          - name: ubuntu-20.04-clang-10-conan
+            os: ubuntu-20.04
+            compiler: clang
+            version: "10"
+            conan: true
+            cmake-args: -DCMAKE_PREFIX_PATH=`pwd`/build -DCMAKE_MODULE_PATH=`pwd`/build -DAUTOBAHN_USE_LIBCXX=NO
+
     steps:
       - uses: actions/checkout@v2
 
@@ -37,10 +50,20 @@ jobs:
       - name: Install
         run: |
           python -m pip install cmake==3.17.3 conan==1.28.1 --upgrade
+
+          wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
+          sudo add-apt-repository 'deb http://apt.llvm.org/focal/ llvm-toolchain-focal-10 main' -y
           sudo apt update
-          sudo apt-get install -y g++-${{ matrix.version }} g++-${{ matrix.version }}-multilib
-          echo "::set-env name=CC::gcc-${{ matrix.version }}"
-          echo "::set-env name=CXX::g++-${{ matrix.version }}"
+
+          if [ "${{ matrix.compiler }}" = "gcc" ]; then
+            sudo apt-get install -y g++-${{ matrix.version }} g++-${{ matrix.version }}-multilib
+            echo "::set-env name=CC::gcc-${{ matrix.version }}"
+            echo "::set-env name=CXX::g++-${{ matrix.version }}"
+          else
+            sudo apt-get install -y clang-${{ matrix.version }} clang-tidy-${{ matrix.version }} g++-multilib
+            echo "::set-env name=CC::clang-${{ matrix.version }}"
+            echo "::set-env name=CXX::clang++-${{ matrix.version }}"
+          fi
 
       - name: Install dependencies (system)
         run: sudo apt-get install -y libboost-all-dev libmsgpack-dev libwebsocketpp-dev

CMakeLists.txt

@@ -2,9 +2,10 @@ cmake_minimum_required(VERSION 2.8)
 
 project(autobahn-cpp)
 
-include(${CMAKE_CURRENT_SOURCE_DIR}/cmake/Includes/CMakeLists.txt)
-
 option(AUTOBAHN_BUILD_EXAMPLES "Build examples" ON)
+option(AUTOBAHN_USE_LIBCXX "Use libc++ instead of libstdc++ when building with Clang" ON)
+
+include(${CMAKE_CURRENT_SOURCE_DIR}/cmake/Includes/CMakeLists.txt)
 
 if(AUTOBAHN_BUILD_EXAMPLES)
     add_subdirectory(examples)

cmake/Includes/CMakeLists.txt

@@ -15,7 +15,10 @@ if(POLICY CMP0054)
 endif()
 
 if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")
-    set(CMAKE_CXX_FLAGS "-stdlib=libc++ -Wno-unused-local-typedefs -Wno-unknown-warning-option ${CMAKE_CXX_FLAGS}")
+    set(CMAKE_CXX_FLAGS "-Wno-unused-local-typedefs -Wno-unknown-warning-option ${CMAKE_CXX_FLAGS}")
+    if(AUTOBAHN_USE_LIBCXX)
+        set(CMAKE_CXX_FLAGS "-stdlib=libc++ ${CMAKE_CXX_FLAGS}")
+    endif()
     if ("${CMAKE_GENERATOR}" STREQUAL "Ninja")
         set(CMAKE_CXX_FLAGS "-fcolor-diagnostics ${CMAKE_CXX_FLAGS}")
     endif()
@@ -57,6 +60,7 @@ find_package(msgpack REQUIRED)
 find_package(websocketpp REQUIRED)
 
 MESSAGE( STATUS "AUTOBAHN_BUILD_EXAMPLES:  " ${AUTOBAHN_BUILD_EXAMPLES} )
+MESSAGE( STATUS "AUTOBAHN_USE_LIBCXX:      " ${AUTOBAHN_USE_LIBCXX} )
 MESSAGE( STATUS "CMAKE_ROOT:               " ${CMAKE_ROOT} )
 MESSAGE( STATUS "CMAKE_INSTALL_PREFIX:     " ${CMAKE_INSTALL_PREFIX} )
 MESSAGE( STATUS "Boost_INCLUDE_DIRS:       " ${Boost_INCLUDE_DIRS} )