Skip to content

Commit

Permalink
Merge pull request #565 from stephanlachnit/p-remove-draft-api-setting
Browse files Browse the repository at this point in the history
Remove ENABLE_DRAFTS option
  • Loading branch information
gummif authored Aug 12, 2024
2 parents 23cd83e + 2fd8bfc commit 6541dd1
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 20 deletions.
15 changes: 7 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
cc: ["gcc-10"]
cxx: ["g++-10"]
drafts: ["ON"]
libzmq: ["4.3.4"]
libzmq: ["4.3.5"]
libzmqbuild: ["cmake"]
include:
# older libzmq and without draft
Expand All @@ -33,15 +33,15 @@ jobs:
cc: "gcc-11"
cxx: "g++-11"
drafts: "OFF"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: "cmake"
# coverage (gcc version should match gcov version)
- os: "ubuntu-20.04"
cppstd: "17"
cc: "gcc-9"
cxx: "g++-9"
drafts: "ON"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: "cmake"
coverage: "-DCOVERAGE=ON"
aptinstall: "lcov"
Expand All @@ -51,15 +51,15 @@ jobs:
cc: "clang-12"
cxx: "clang++-12"
drafts: "ON"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: "cmake"
# macos
- os: "macos-latest"
cppstd: "17"
cc: "clang"
cxx: "clang++"
drafts: "OFF"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: false
brewinstall: "zeromq"
# windows
Expand All @@ -68,15 +68,15 @@ jobs:
cc: "msbuild"
cxx: "msbuild"
drafts: "ON"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: "cmake"
platform: "-Ax64"
- os: "windows-2022"
cppstd: "20"
cc: "msbuild"
cxx: "msbuild"
drafts: "ON"
libzmq: "4.3.4"
libzmq: "4.3.5"
libzmqbuild: "cmake"
platform: "-Ax64"

Expand Down Expand Up @@ -132,7 +132,6 @@ jobs:
run: |
cmake -H. -Bbuild ${{ matrix.platform}} ${{ matrix.coverage }} \
-DCMAKE_BUILD_TYPE=${BUILDTYPE} \
-DENABLE_DRAFTS=${{ matrix.drafts }} \
-DCMAKE_CXX_STANDARD=${{ matrix.cppstd }}
cmake --build build --config ${BUILDTYPE} -j ${THREADS}
echo "CPPZMQ=${PWD}/build" >> ${GITHUB_ENV}
Expand Down
12 changes: 0 additions & 12 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -26,18 +26,6 @@ if (NOT TARGET libzmq AND NOT TARGET libzmq-static)
endif()
endif()

if (EXISTS "${CMAKE_SOURCE_DIR}/.git")
OPTION (ENABLE_DRAFTS "Build and install draft classes and methods" ON)
else ()
OPTION (ENABLE_DRAFTS "Build and install draft classes and methods" OFF)
endif ()
if (ENABLE_DRAFTS)
ADD_DEFINITIONS (-DZMQ_BUILD_DRAFT_API)
set (pkg_config_defines "-DZMQ_BUILD_DRAFT_API=1")
else (ENABLE_DRAFTS)
set (pkg_config_defines "")
endif (ENABLE_DRAFTS)

message(STATUS "cppzmq v${cppzmq_VERSION}")

set(CPPZMQ_HEADERS
Expand Down

0 comments on commit 6541dd1

Please sign in to comment.