Add support for g++-13,14

Summary: related to T13767, #9 fix compilation error on g++-14 add g++-13 & g++-14 workflow to CI (requires ubuntu 24.04) Reviewers: ivica Reviewed By: ivica Subscribers: miljen, iljazovic Differential Revision: https://repo.mireo.local/D29794
mireo · Jun 3, 2024 · d52090f · d52090f
1 parent 927c1c6
commit d52090f
Show file tree

Hide file tree

Showing 7 changed files with 31 additions and 8 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -15,7 +15,7 @@ env:
 
 jobs:
   windows:
-    name: "${{ matrix.toolset }} std=c++${{ matrix.cxxstd }} ${{ matrix.build-type }}"
+    name: "${{ matrix.toolset }} std=c++${{ matrix.cxxstd }}"
     defaults:
       run:
         shell: cmd
@@ -114,7 +114,7 @@ jobs:
         .\\test\\build\\${{ matrix.build-type }}\\mqtt-test.exe
 
   posix:
-    name: "${{ matrix.toolset }} std=c++${{ matrix.cxxstd }} ${{ matrix.build-type }}"
+    name: "${{ matrix.toolset }} std=c++${{ matrix.cxxstd }} ${{ matrix.container }}"
 
     defaults:
       run:
@@ -150,6 +150,26 @@ jobs:
             os: ubuntu-latest
             container: ubuntu:22.04
             build-type: 'Release'
+            cxxstd: 17
+            cxxflags: ''
+            ldflags: ''
+
+          - toolset: g++-13
+            compiler: g++-13
+            install: g++-13
+            os: ubuntu-24.04
+            container: ubuntu:24.04
+            build-type: 'Release'
+            cxxstd: 17
+            cxxflags: ''
+            ldflags: ''
+
+          - toolset: g++-14
+            compiler: g++-14
+            install: g++-14
+            os: ubuntu-24.04
+            container: ubuntu:24.04
+            build-type: 'Release'
             cxxstd: 20
             cxxflags: ''
             ldflags: ''

diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -15,7 +15,7 @@ env:
 
 jobs:
   posix:
-    name: "coverage ${{ matrix.compiler }} -std=c++${{ matrix.cxxstd }}"
+    name: "coverage ${{ matrix.compiler }} -std=c++${{ matrix.cxxstd }} ${{ matrix.container }}"
     defaults:
       run:
         shell: bash

diff --git a/README.md b/README.md
@@ -128,8 +128,8 @@ Async.MQTT5 is a header-only library. To use Async.MQTT5 it requires the followi
 - **OpenSSL**. Only if you require an SSL connection by using [boost::asio::ssl::stream](https://www.boost.org/doc/libs/1_82_0/doc/html/boost_asio/reference/ssl__stream.html).
 
 Async.MQTT5 has been tested with the following compilers: 
-- clang 12.0, 13.0, 14.0, 15.0 (Linux)
-- GCC 9, 10, 11, 12 (Linux)
+- clang 12.0 - 15.0 (Linux)
+- GCC 9 - 14 (Linux)
 - MSVC 14.37 - Visual Studio 2022 (Windows)
 
 Contributing

diff --git a/doc/qbk/01_intro.qbk b/doc/qbk/01_intro.qbk
@@ -127,8 +127,8 @@ To use __Self__ it requires the following:
 
 __Self__ has been tested with the following compilers:
 
-* clang 12.0, 13.0, 14.0, 15.0 (Linux)
-* GCC 9, 10, 11, 12 (Linux)
+* clang 12.0 - 15.0 (Linux)
+* GCC 9 - 14 (Linux)
 * MSVC 14.37 - Visual Studio 2022 (Windows)
 
 [heading Acknowledgements]

diff --git a/include/async_mqtt5/detail/control_packet.hpp b/include/async_mqtt5/detail/control_packet.hpp
@@ -8,6 +8,7 @@
 #ifndef ASYNC_MQTT5_CONTROL_PACKET_HPP
 #define ASYNC_MQTT5_CONTROL_PACKET_HPP
 
+#include <algorithm>
 #include <vector>
 
 #include <boost/smart_ptr/allocate_unique.hpp>

diff --git a/include/async_mqtt5/impl/re_auth_op.hpp b/include/async_mqtt5/impl/re_auth_op.hpp
@@ -11,6 +11,7 @@
 #include <boost/asio/detached.hpp>
 
 #include <async_mqtt5/error.hpp>
+#include <async_mqtt5/reason_codes.hpp>
 
 #include <async_mqtt5/detail/control_packet.hpp>
 #include <async_mqtt5/detail/any_authenticator.hpp>

diff --git a/test/include/test_common/packet_util.hpp b/test/include/test_common/packet_util.hpp
@@ -8,8 +8,9 @@
 #ifndef ASYNC_MQTT5_TEST_PACKET_UTIL_HPP
 #define ASYNC_MQTT5_TEST_PACKET_UTIL_HPP
 
-#include <string>
 #include <bitset>
+#include <sstream>
+#include <string>
 
 #include <boost/algorithm/string/join.hpp>