Skip to content

Commit c1a758e

Browse files
committed
Unite cpack
1 parent 9cbe26f commit c1a758e

File tree

5 files changed

+29
-42
lines changed

5 files changed

+29
-42
lines changed

.github/workflows/release-test.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ jobs:
124124
git config --global --add safe.directory /__w/CCF/CCF
125125
mkdir build
126126
cd build
127-
CC=`which clang` CXX=`which clang++` cmake -GNinja -DCOMPILE_TARGET=${{ matrix.platform.name }} -DCLIENT_PROTOCOLS_TEST=ON -DPACKAGE_TYPE=${{ matrix.platform.package_format }} -DCMAKE_BUILD_TYPE=Release ..
127+
CC=`which clang` CXX=`which clang++` cmake -GNinja -DCOMPILE_TARGET=${{ matrix.platform.name }} -DCLIENT_PROTOCOLS_TEST=ON -DCMAKE_BUILD_TYPE=Release ..
128128
ninja -v | tee build.log
129129
130130
- name: "Install Extended Testing Tools"

.github/workflows/release.yml

+1-5
Original file line numberDiff line numberDiff line change
@@ -70,21 +70,17 @@ jobs:
7070
- name: virtual
7171
os: ubuntu
7272
image: ghcr.io/microsoft/ccf/ci/default:build-14-01-2025
73-
package_format: DEB
7473
test_filter: "benchmark|unit"
7574
- name: snp
7675
os: ubuntu
7776
image: ghcr.io/microsoft/ccf/ci/default:build-14-01-2025
78-
package_format: DEB
7977
- name: virtual
8078
os: azure-linux
8179
image: mcr.microsoft.com/azurelinux/base/core:3.0
82-
package_format: RPM
8380
test_filter: "benchmark|unit|protocolstest|lts"
8481
- name: snp
8582
os: azure-linux
8683
image: mcr.microsoft.com/azurelinux/base/core:3.0
87-
package_format: RPM
8884

8985
runs-on: [self-hosted, 1ES.Pool=gha-virtual-ccf-sub]
9086
container:
@@ -118,7 +114,7 @@ jobs:
118114
git config --global --add safe.directory /__w/CCF/CCF
119115
mkdir build
120116
cd build
121-
CC=`which clang` CXX=`which clang++` cmake -GNinja -DCOMPILE_TARGET=${{ matrix.platform.name }} -DCLIENT_PROTOCOLS_TEST=ON -DPACKAGE_TYPE=${{ matrix.platform.package_format }} -DCMAKE_BUILD_TYPE=Release ..
117+
CC=`which clang` CXX=`which clang++` cmake -GNinja -DCOMPILE_TARGET=${{ matrix.platform.name }} -DCLIENT_PROTOCOLS_TEST=ON -DCMAKE_BUILD_TYPE=Release ..
122118
ninja -v | tee build.log
123119
124120
- name: "Install Extended Testing Tools"

CMakeLists.txt

+1-5
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,7 @@ if(CMAKE_INSTALL_PREFIX_INITIALIZED_TO_DEFAULT)
3636
)
3737
endif()
3838

39-
if(PACKAGE_TYPE STREQUAL "DEB")
40-
include(${CCF_DIR}/cmake/cpack_deb.cmake)
41-
elseif(PACKAGE_TYPE STREQUAL "RPM")
42-
include(${CCF_DIR}/cmake/cpack_rpm.cmake)
43-
endif()
39+
include(${CCF_DIR}/cmake/cpack_settings.cmake)
4440

4541
message(STATUS "CMAKE_INSTALL_PREFIX is '${CMAKE_INSTALL_PREFIX}'")
4642

cmake/cpack_rpm.cmake

-31
This file was deleted.

cmake/cpack_deb.cmake cmake/cpack_settings.cmake

+26
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ set(CPACK_RESOURCE_FILE_LICENSE "${CCF_DIR}/LICENSE")
88
set(CPACK_PACKAGE_VERSION ${CCF_RELEASE_VERSION})
99
set(CPACK_PACKAGING_INSTALL_PREFIX ${CMAKE_INSTALL_PREFIX})
1010

11+
# DEB-specific settings
12+
1113
set(CPACK_DEBIAN_PACKAGE_VERSION "${CCF_RELEASE_VERSION}")
1214

1315
if(CCF_VERSION_SUFFIX)
@@ -32,6 +34,30 @@ endif()
3234

3335
list(JOIN CCF_DEB_DEPENDENCIES ", " CPACK_DEBIAN_PACKAGE_DEPENDS)
3436

37+
message(STATUS "DEB package dependencies: ${CCF_DEB_DEPENDENCIES}")
38+
3539
set(CPACK_DEBIAN_FILE_NAME DEB-DEFAULT)
3640

41+
# RPM-specific settings
42+
43+
set(CPACK_RPM_PACKAGE_VERSION "${CCF_RELEASE_VERSION}")
44+
45+
if(CCF_VERSION_SUFFIX)
46+
set(CPACK_RPM_PACKAGE_VERSION
47+
"${CPACK_RPM_PACKAGE_VERSION}~${CCF_VERSION_SUFFIX}"
48+
)
49+
endif()
50+
51+
message(STATUS "RPM package version: ${CPACK_RPM_PACKAGE_VERSION}")
52+
53+
set(CCF_RPM_DEPENDENCIES
54+
"libuv >= 1.34.2, openssl >= 3.3.0, nghttp2 >= 1.40.0, curl >= 7.68.0, libcxxabi >= 18.1.2"
55+
)
56+
57+
message(STATUS "RPM package dependencies: ${CCF_RPM_DEPENDENCIES}")
58+
59+
set(CPACK_RPM_PACKAGE_REQUIRES "${CCF_RPM_DEPENDENCIES}")
60+
61+
set(CPACK_RPM_FILE_NAME RPM-DEFAULT)
62+
3763
include(CPack)

0 commit comments

Comments
 (0)