Skip to content

Commit 255b284

Browse files
committed
Fix Coverity warnings
Signed-off-by: Raul Metsma <[email protected]>
1 parent 1cab9e2 commit 255b284

File tree

3 files changed

+21
-22
lines changed

3 files changed

+21
-22
lines changed

.github/workflows/cmake-macos.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,9 @@ jobs:
6565
- name: Install Qt
6666
uses: jurplel/install-qt-action@v4
6767
with:
68-
version: 6.10.0
68+
version: 6.10.1
6969
arch: clang_64
70+
cache: true
7071

7172
- name: Configure
7273
run: cmake -DCMAKE_BUILD_TYPE=${BUILD_TYPE} -DCMAKE_OSX_ARCHITECTURES="arm64;x86_64" -B ${BUILD_DIR} -S .
@@ -85,5 +86,5 @@ jobs:
8586
with:
8687
name: web-eid-app-macos-build-${{github.run_number}}
8788
path: |
88-
build/*/*/*.pkg
89-
build/*/*/*.dmg
89+
build/src/*/*.pkg
90+
build/src/*/*.dmg

.github/workflows/cmake-windows.yml

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,12 @@ env:
66
BUILD_TYPE: RelWithDebInfo
77
BUILD_NUMBER: ${{github.run_number}}
88
CMAKE_BUILD_PARALLEL_LEVEL: 4
9-
VCPKG_INSTALLED_DIR: ${{ github.workspace }}/build/vcpkg_installed
109
# Use libelectronic-id vcpkg manifest for dependencies
1110
VCPKG_MANIFEST_DIR: ./lib/libelectronic-id/.github
1211

1312
jobs:
1413
build:
15-
runs-on: windows-2025
14+
runs-on: ${{ matrix.arch == 'arm64' && 'windows-11-arm' || 'windows-2025' }}
1615
strategy:
1716
matrix:
1817
arch: [x64, arm64]
@@ -25,42 +24,41 @@ jobs:
2524
persist-credentials: false
2625

2726
- name: Cache vcpkg
27+
id: cache
2828
uses: actions/cache@v4
2929
with:
30-
path: ${{ github.workspace }}/vcpkg_cache
30+
path: |
31+
${{ github.workspace }}/vcpkg_cache
32+
~/.dotnet/tools
33+
~/.wix
3134
key: vcpkg-${{ matrix.arch }}-${{ hashFiles(format('{0}/vcpkg.json', env.VCPKG_MANIFEST_DIR)) }}
3235

33-
- name: Prepare vcpkg and libraries
34-
uses: lukka/run-vcpkg@v11
35-
with:
36-
vcpkgJsonGlob: ${{ env.VCPKG_MANIFEST_DIR }}/vcpkg.json
37-
runVcpkgInstall: true
38-
env:
39-
VCPKG_BINARY_SOURCES: clear;files,${{ github.workspace }}/vcpkg_cache,readwrite
40-
VCPKG_DEFAULT_TRIPLET: ${{ matrix.arch }}-windows
41-
4236
- name: Install Qt
4337
uses: jurplel/install-qt-action@v4
4438
with:
45-
version: 6.9.3
46-
arch: ${{ matrix.arch == 'arm64' && 'win64_msvc2022_arm64_cross_compiled' || 'win64_msvc2022_64' }}
39+
version: 6.10.1
40+
arch: ${{ matrix.arch == 'arm64' && 'win64_msvc2022_arm64' || 'win64_msvc2022_64' }}
41+
cache: true
4742

4843
- name: Setup MS Visual C++ dev env
4944
uses: ilammy/msvc-dev-cmd@v1
5045
with:
51-
arch: ${{ matrix.arch == 'arm64' && 'amd64_arm64' || 'amd64' }}
46+
arch: ${{ matrix.arch }}
5247

5348
- name: Install WiX
49+
if: steps.cache.outputs.cache-hit != 'true'
5450
run: |
5551
dotnet tool install --global wix --version 6.0.2
5652
wix extension -g add WixToolset.UI.wixext/6.0.2
5753
wix extension -g add WixToolset.Util.wixext/6.0.2
5854
wix extension -g add WixToolset.BootstrapperApplications.wixext/6.0.2
5955
6056
- name: Configure
57+
env:
58+
VCPKG_BINARY_SOURCES: clear;files,${{ github.workspace }}/vcpkg_cache,readwrite
6159
run: |
6260
cmake "-GNinja" -S . -B build `
63-
"-DCMAKE_TOOLCHAIN_FILE=${env:RUNVCPKG_VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake" `
61+
"-DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake" `
6462
"-DCMAKE_BUILD_TYPE=${env:BUILD_TYPE}" `
6563
"-DVCPKG_MANIFEST_DIR=${{ env.VCPKG_MANIFEST_DIR }}"
6664
@@ -79,6 +77,6 @@ jobs:
7977
with:
8078
name: web-eid-app-windows-build-${{matrix.arch}}-${{github.run_number}}
8179
path: |
82-
build/src/app/*.msi
83-
build/src/app/*.exe
80+
build/src/*/*.msi
81+
build/src/*/*.exe
8482
build/**/*.pdb

0 commit comments

Comments
 (0)