Merge pull request #1176 from redboltz/update_to_cpp_8.0.0

Update the version to 8.0.0

.github/depends/boost.sh

@@ -49,7 +49,7 @@ while getopts "b:t:p:" c; do
 done
 
 mkdir $prefix || exit 1
-wget https://boostorg.jfrog.io/artifactory/main/release/1.85.0/source/boost_1_85_0.tar.bz2 || exit 1
+wget https://archives.boost.io/release/1.85.0/source/boost_1_85_0.tar.bz2 || exit 1
 tar xf boost_1_85_0.tar.bz2 || exit 1
 cd boost_1_85_0
 ./bootstrap.sh || exit 1

.github/depends/zlib.sh

@@ -27,9 +27,9 @@ while getopts "b:t:p:" c; do
 done
 
 mkdir $prefix || exit 1
-wget https://zlib.net/zlib-1.3.1.tar.gz || exit 1
+wget https://zlib.net/zlib-1.3.2.tar.gz || exit 1
-tar -xf zlib-1.3.1.tar.gz || exit 1
+tar -xf zlib-1.3.2.tar.gz || exit 1
-cd zlib-1.3.1
+cd zlib-1.3.2
 
 build()
 {

.github/workflows/coverage.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v1
+    - uses: actions/checkout@v3
     - name: Install build dependencies
       run: |
         sudo apt-get update
@@ -24,7 +24,7 @@ jobs:
 
     - name: Cache boost
       id: cache-boost
-      uses: actions/cache@v1
+      uses: actions/cache@v3
       with:
         path: ~/boost-prefix/
         key: ${{ runner.os }}-boost-1-85-0-2024-05-27
@@ -35,7 +35,7 @@ jobs:
 
     - name: Cache zlib
       id: cache-zlib
-      uses: actions/cache@v1
+      uses: actions/cache@v3
       with:
         path: ~/zlib-prefix/
         key: ${{ runner.os }}-zlib-64-1-2-11-2021-08-09
@@ -63,12 +63,15 @@ jobs:
         cmake --build build --target all || exit 1
         ctest --test-dir build || exit 1
 
-    - name: Upload coverage to Codecov
+    - name: Generate coverage
       working-directory: build
       run: |
         # Create lcov report
-        lcov --capture --directory . --output-file coverage.info
+        lcov --capture --directory . --output-file coverage.info --ignore-errors mismatch
         lcov --remove coverage.info '/usr/*' --output-file coverage.info # filter system-files
         lcov --list coverage.info # debug info
-        # Uploading report to CodeCov
+    - name: Upload coverage to Codecov
-        bash <(curl -s https://codecov.io/bash) -f coverage.info || echo "Codecov did not collect coverage reports"
+      uses: codecov/codecov-action@v5
+      with:
+        files: build/coverage.info
+        token: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/gha.yml

@@ -84,7 +84,7 @@ jobs:
 
   linux:
     name: ${{ format('Linux (pattern {0})', matrix.pattern) }}
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-24.04
     strategy:
       fail-fast: false
       matrix:
@@ -100,34 +100,44 @@ jobs:
         sudo apt-get install g++-10-multilib -y  # for 32-bit compile
         ./ci/set_gcc_10.sh
 
+    - name: Set Architecture Environment
+      shell: bash
+      run: |
+        if [ "${{ matrix.pattern }}" = "5" ] || [ "${{ matrix.pattern }}" = "11" ]; then
+          echo "ARCH=32" >> $GITHUB_ENV
+          echo "BOOST_BIT=both" >> $GITHUB_ENV
+        else
+          echo "ARCH=64" >> $GITHUB_ENV
+          echo "BOOST_BIT=64" >> $GITHUB_ENV
+        fi
+        
     - name: Cache boost
       id: cache-boost
       uses: actions/cache@v3
       with:
         path: ~/boost-prefix/
-        key: ${{ runner.os }}-boost-1-85-0-2024-05-27
+        key: ${{ runner.os }}-boost-1-85-0-2026-05-30-${{ env.ARCH }}
 
     - name: Build boost
       if: steps.cache-boost.outputs.cache-hit != 'true'
-      run: ./.github/depends/boost.sh -b both -t gcc -p $HOME/boost-prefix
+      run: ./.github/depends/boost.sh -b ${{ env.BOOST_BIT }} -t gcc -p $HOME/boost-prefix
 
     - name: Cache zlib
       id: cache-zlib
       uses: actions/cache@v3
       with:
         path: ~/zlib-prefix/
-        key: ${{ runner.os }}-zlib-1-2-13-2022-11-02
+        key: ${{ runner.os }}-zlib-1-3-2-2026-05-30-${{ env.ARCH }}
 
     - name: Build zlib
       if: steps.cache-zlib.outputs.cache-hit != 'true'
-      run: ./.github/depends/zlib.sh -b both -p $HOME/zlib-prefix
+      run: ./.github/depends/zlib.sh -b ${{ env.BOOST_BIT }} -p $HOME/zlib-prefix
 
     - name: Build and test
       shell: bash
       run: |
         # default configuration - overwrite its params later depending on matrix.pattern
         export MSGPACK_CXX_VERSION="MSGPACK_CXX20=ON"
-        export ARCH=64
         export API_VERSION=3
         export CHAR_SIGN="signed"
         export X3_PARSE="OFF"
@@ -136,7 +146,7 @@ jobs:
 
         case ${{ matrix.pattern }} in
           0)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export MSGPACK_CXX_VERSION="MSGPACK_CXX11=ON"
             ;;
           1)
@@ -144,7 +154,7 @@ jobs:
             export MSGPACK_CXX_VERSION="MSGPACK_CXX11=ON"
             ;;
           2)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export MSGPACK_CXX_VERSION="MSGPACK_CXX14=ON"
             ;;
           3)
@@ -153,16 +163,16 @@ jobs:
             export MSGPACK_USE_STD_VARIANT_ADAPTOR="MSGPACK_USE_STD_VARIANT_ADAPTOR=ON"
             ;;
           4)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export MSGPACK_CXX_VERSION="MSGPACK_CXX20=ON"
             export NO_BOOST="-DMSGPACK_NO_BOOST"
             ;;
           5)
+            # ARCH=32
             export CXX="g++-10"
-            export ARCH=32
             ;;
           6)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export API_VERSION=2
             ;;
           7)
@@ -170,7 +180,7 @@ jobs:
             export API_VERSION=1
             ;;
           8)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export CHAR_SIGN="unsigned"
             ;;
           9)
@@ -178,12 +188,12 @@ jobs:
             export X3_PARSE="ON"
             ;;
           10)
-            export CXX="clang++-10"
+            export CXX="clang++"
             export ACTION="ci/build_regression.sh"
             ;;
           11)
+            # ARCH=32
             export CXX="g++-10"
-            export ARCH=32
             export CHAR_SIGN="unsigned"
             export X3_PARSE="ON"
             ;;
@@ -195,11 +205,11 @@ jobs:
 
   windows:
     name: ${{ format('Windows cxx{0}', matrix.cxx) }}
-    runs-on: windows-2019
+    runs-on: windows-2022
     strategy:
       fail-fast: false
       matrix:
-        # MSVC2019 only supports /std:c++14, /std:c++17 and /std:c++latest
+        # MSVC2022 only supports /std:c++14, /std:c++17 and /std:c++latest
         cxx: [14, 17, 20]
         pp_flag: ["/Zc:preprocessor-", "/Zc:preprocessor"]
     steps:
@@ -228,7 +238,7 @@ jobs:
         md build
         cmake `
             -A x64 `
-            -G "Visual Studio 16 2019" `
+            -G "Visual Studio 17 2022" `
             -D CMAKE_TOOLCHAIN_FILE="C:/vcpkg/scripts/buildsystems/vcpkg.cmake" `
             -D MSGPACK_BUILD_TESTS=ON `
             -D $CPPVER `

CHANGELOG.md

@@ -1,3 +1,14 @@
+# 2026-05-30 version 8.0.0
+  * Add old style find boost applying option to cmake. (#1172)
+  * Add missing include type_traits (#1162)
+  * Remove duplicated include (#1156, #1157)
+  * Fix MSVC C++20 modules error for nested msgpack::object::with_zone (#1154)
+  * Refine CI (#1150, #1151, #1163)
+  * Refine documents (#1146, #1149)
+
+## << breaking changes >>
+  * Update cmake minimum required to 3.5-4.0 (#1159)
+
 # 2024-11-02 version 7.0.0
   * Refine build system (#1133)
   * Refine CI (#1122)

CMakeLists.txt

@@ -1,12 +1,4 @@
-if(${CMAKE_VERSION} VERSION_GREATER "3.4")
+CMAKE_MINIMUM_REQUIRED (VERSION 3.5...4.0)
-    CMAKE_MINIMUM_REQUIRED (VERSION 3.5)
-else()
-    CMAKE_MINIMUM_REQUIRED (VERSION 2.8.12)
-    IF ((CMAKE_VERSION VERSION_GREATER 3.1) OR
-        (CMAKE_VERSION VERSION_EQUAL 3.1))
-        CMAKE_POLICY(SET CMP0054 NEW)
-    ENDIF ()
-endif()
 
 PROJECT (msgpack-cxx LANGUAGES CXX)
 
@@ -75,8 +67,10 @@ ENDIF ()
 IF (MSGPACK_USE_BOOST)
     IF ((CMAKE_VERSION VERSION_GREATER 3.30) OR
         (CMAKE_VERSION VERSION_EQUAL 3.30))
+        IF (NOT MSGPACK_BOOST_USE_OLD_CMAKE)
             CMAKE_POLICY(SET CMP0167 NEW)
         ENDIF ()
+    ENDIF ()
     SET (Boost_USE_MULTITHREADED ON)
 
     IF (MSGPACK_USE_STATIC_BOOST)
@@ -164,10 +158,10 @@ IF (MSGPACK_BUILD_TESTS)
         MESSAGE(FATAL_ERROR "Test requires -DMSGPACK_USE_BOOST=ON")
     ENDIF ()
     ENABLE_TESTING ()
+    INCLUDE (CTest)
     # MEMORYCHECK_COMMAND_OPTIONS needs to place prior to CTEST_MEMORYCHECK_COMMAND
     SET (MEMORYCHECK_COMMAND_OPTIONS "--leak-check=full --show-leak-kinds=definite,possible --error-exitcode=1")
     FIND_PROGRAM (CTEST_MEMORYCHECK_COMMAND NAMES valgrind)
-    INCLUDE (Dart)
     ADD_SUBDIRECTORY (test)
 ENDIF ()
 

README.md

@@ -1,8 +1,8 @@
 `msgpack` for C++
 ===================
 
-Version 7.0.0 [![Build Status](https://github.com/msgpack/msgpack-c/workflows/CI/badge.svg?branch=cpp_master)](https://github.com/msgpack/msgpack-c/actions) [![Build status](https://ci.appveyor.com/api/projects/status/8kstcgt79qj123mw/branch/cpp_master?svg=true)](https://ci.appveyor.com/project/redboltz/msgpack-c/branch/cpp_master)
+Version 8.0.0 [![Build Status](https://github.com/msgpack/msgpack-c/workflows/CI/badge.svg?branch=cpp_master)](https://github.com/msgpack/msgpack-c/actions) [![Build status](https://ci.appveyor.com/api/projects/status/8kstcgt79qj123mw/branch/cpp_master?svg=true)](https://ci.appveyor.com/project/redboltz/msgpack-c/branch/cpp_master)
-[![codecov](https://codecov.io/gh/msgpack/msgpack-c/branch/cpp_master/graph/badge.svg)](https://codecov.io/gh/msgpack/msgpack-c/branch/cpp_master)
+[![codecov](https://codecov.io/gh/msgpack/msgpack-c/branch/cpp_master/graph/badge.svg)](https://app.codecov.io/gh/msgpack/msgpack-c/tree/cpp_master)
 
 It's like JSON but smaller and faster.
 
@@ -87,7 +87,6 @@ For C++ compiler
 clang++ -DMSGPACK_NO_BOOST your_code.cpp
 ```
 
-
 Usage
 -----
 
@@ -96,7 +95,7 @@ Usage
 
   ```cmake
   # ...
-  find_package(msgpack REQUIRED)
+  find_package(msgpack-cxx REQUIRED)
   # ...
   target_link_libraries(your_target_name <PRIVATE/PUBLIC/INTERFACE> msgpack-cxx)
   # ...

appveyor.yml

@@ -1,4 +1,4 @@
-version: 7.0.0.{build}
+version: 8.0.0.{build}
 
 branches:
   only:
@@ -13,10 +13,10 @@ environment:
       boost_subdir: lib32-msvc-14.0
 build_script:
   - ps: |
-      appveyor DownloadFile http://zlib.net/zlib-1.3.1.tar.gz -FileName zlib-1.3.1.tar.gz
+      appveyor DownloadFile http://zlib.net/zlib-1.3.2.tar.gz -FileName zlib-1.3.2.tar.gz
-      7z x zlib-1.3.1.tar.gz 2> $null
+      7z x zlib-1.3.2.tar.gz 2> $null
-      7z x zlib-1.3.1.tar 2> $null
+      7z x zlib-1.3.2.tar 2> $null
-      cd zlib-1.3.1
+      cd zlib-1.3.2
 
       md build
       md prefix
@@ -24,7 +24,7 @@ build_script:
 
       cmake `
           -G $env:msvc `
-          -D CMAKE_INSTALL_PREFIX="$env:APPVEYOR_BUILD_FOLDER\zlib-1.3.1\prefix" `
+          -D CMAKE_INSTALL_PREFIX="$env:APPVEYOR_BUILD_FOLDER\zlib-1.3.2\prefix" `
           ..
       if ($LastExitCode -ne 0) { exit $LastExitCode }
 
@@ -41,8 +41,9 @@ build_script:
           $env:cpp11 `
           -D MSGPACK_BUILD_EXAMPLES=ON `
           -D MSGPACK_BUILD_TESTS=ON `
+          -D MSGPACK_BOOST_USE_OLD_CMAKE=ON `
           -D CMAKE_EXE_LINKER_FLAGS=/LIBPATH:"$env:boost_prefix\$env:boost_subdir" `
-          -D CMAKE_PREFIX_PATH="$env:boost_prefix;$env:APPVEYOR_BUILD_FOLDER\zlib-1.3.1\prefix" `
+          -D CMAKE_PREFIX_PATH="$env:boost_prefix;$env:APPVEYOR_BUILD_FOLDER\zlib-1.3.2\prefix" `
           -D CMAKE_INSTALL_PREFIX="$env:APPVEYOR_BUILD_FOLDER\prefix" `
           -D CMAKE_CXX_FLAGS="/D_VARIADIC_MAX=10 /EHsc /DBOOST_ALL_DYN_LINK" `
           ..
@@ -52,5 +53,5 @@ build_script:
       if ($LastExitCode -ne 0) { exit $LastExitCode }
 
 test_script:
-- set PATH=%PATH%;%APPVEYOR_BUILD_FOLDER%\zlib-1.3.1\build\Release;%APPVEYOR_BUILD_FOLDER%\build\release;%boost_prefix%\%boost_subdir%
+- set PATH=%PATH%;%APPVEYOR_BUILD_FOLDER%\zlib-1.3.2\build\Release;%APPVEYOR_BUILD_FOLDER%\build\release;%boost_prefix%\%boost_subdir%
 - ctest -VV -C Release

cmake/CodeCoverage.cmake

@@ -48,7 +48,7 @@ FUNCTION(SETUP_TARGET_FOR_COVERAGE _targetname _testrunner _outputname)
 
     # Show info where to find the report
     ADD_CUSTOM_COMMAND(TARGET ${_targetname} POST_BUILD
-        COMMAND ;
+        COMMAND ${CMAKE_COMMAND} -E echo ""
         COMMENT "Open ./${_outputname}/index.html in your browser to view the coverage report."
     )
 

codecov.yml

@@ -31,6 +31,6 @@ ignore:
   - "erb"
   - "ci"
   - "cmake"
-  - "examle"
+  - "example"
   - "external"
   - "usr"

include/msgpack/v1/adaptor/cpp11/timespec.hpp

@@ -13,7 +13,6 @@
 #include "msgpack/versioning.hpp"
 #include "msgpack/adaptor/adaptor_base.hpp"
 #include "msgpack/object.hpp"
-#include "msgpack/object.hpp"
 
 #include <ctime>
 

include/msgpack/v1/cpp_config_decl.hpp

@@ -87,6 +87,7 @@ struct is_pointer;
 
 #include <memory>
 #include <tuple>
+#include <type_traits>
 
 namespace msgpack {
 /// @cond

include/msgpack/v1/fbuffer.hpp

@@ -31,6 +31,7 @@ public:
     {
         MSGPACK_ASSERT(buf || len == 0);
         if (!buf) return;
+        if (len == 0) return;
         if (1 != fwrite(buf, len, 1, m_file)) {
             throw std::runtime_error("fwrite() failed");
         }

include/msgpack/v1/object.hpp

@@ -32,11 +32,11 @@ struct object_kv {
     msgpack::object val;
 };
 
-struct object::with_zone : msgpack::object {
+struct object_with_zone_type : msgpack::object {
-    with_zone(msgpack::zone& z) : zone(z) { }
+    object_with_zone_type(msgpack::zone& z) : zone(z) { }
     msgpack::zone& zone;
 private:
-    with_zone();
+    object_with_zone_type();
 };
 
 

include/msgpack/v1/object_fwd.hpp

@@ -68,6 +68,8 @@ public:
 
 #endif // !defined(MSGPACK_USE_CPP03)
 
+struct object_with_zone_type;
+
 /// Object class that corresponding to MessagePack format object
 /**
  * See https://github.com/msgpack/msgpack-c/wiki/v1_1_cpp_object
@@ -219,7 +221,8 @@ struct object {
     template <typename T>
     object& operator=(const T& v);
 
-    struct with_zone;
+    // Not a nested struct (i.e. 'struct with_zone;') to work around MSVC C++20 modules error C2504
+    typedef object_with_zone_type with_zone;
 
 protected:
     struct implicit_type;

include/msgpack/version_master.hpp

@@ -1,3 +1,3 @@
-#define MSGPACK_VERSION_MAJOR    7
+#define MSGPACK_VERSION_MAJOR    8
 #define MSGPACK_VERSION_MINOR    0
 #define MSGPACK_VERSION_REVISION 0

test-install/CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.1 FATAL_ERROR)
+cmake_minimum_required(VERSION 3.5...4.0)
 
 project(test-install LANGUAGES CXX)
 

test/buffer.cpp

@@ -90,6 +90,7 @@ BOOST_AUTO_TEST_CASE(fbuffer)
     fbuf.write("a", 1);
     fbuf.write("a", 1);
     fbuf.write("a", 1);
+    fbuf.write("", 0);
 
     fflush(file);
     rewind(file);