Skip to content

Commit cd689f4

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

File tree

5 files changed

+28
-24
lines changed

5 files changed

+28
-24
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/app/*.pkg
90+
build/src/app/*.dmg

.github/workflows/cmake-windows.yml

Lines changed: 21 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
11-
VCPKG_MANIFEST_DIR: ./lib/libelectronic-id/.github
10+
VCPKG_MANIFEST_DIR: ./lib/libelectronic-id
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
@@ -81,4 +79,9 @@ jobs:
8179
path: |
8280
build/src/app/*.msi
8381
build/src/app/*.exe
84-
build/**/*.pdb
82+
83+
- name: Upload debug artifacts
84+
uses: actions/upload-artifact@v5
85+
with:
86+
name: web-eid-app-windows-debug-${{matrix.arch}}-${{github.run_number}}
87+
path: build/**/*.pdb

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -332,7 +332,7 @@ Use _Powershell_ to run the following commands to build the project.
332332
"-DCMAKE_PREFIX_PATH=${QT_ROOT}" `
333333
"-DCMAKE_TOOLCHAIN_FILE=${VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake" `
334334
"-DCMAKE_BUILD_TYPE=${BUILD_TYPE}" `
335-
"-DVCPKG_MANIFEST_DIR=lib/libelectronic-id/.github"
335+
"-DVCPKG_MANIFEST_DIR=lib/libelectronic-id"
336336

337337
- Run the build and installer build:
338338

build.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@ param(
66
[string]$arch = "x64"
77
)
88

9-
& $cmake -S . -B build\windows -A $arch -DCMAKE_BUILD_TYPE=$buildtype "-DCMAKE_PREFIX_PATH=$qtdir" "-DCMAKE_TOOLCHAIN_FILE=$vcpkgroot\scripts\buildsystems\vcpkg.cmake" "-DVCPKG_MANIFEST_DIR=lib/libelectronic-id/.github"
9+
& $cmake -S . -B build\windows -A $arch -DCMAKE_BUILD_TYPE=$buildtype "-DCMAKE_PREFIX_PATH=$qtdir" "-DCMAKE_TOOLCHAIN_FILE=$vcpkgroot\scripts\buildsystems\vcpkg.cmake" "-DVCPKG_MANIFEST_DIR=lib/libelectronic-id"
1010
& $cmake --build build\windows --config $buildtype

0 commit comments

Comments
 (0)