Skip to content

Commit 570fab1

Browse files
committed
fix: broken ci compilation for Windows
peflags.exe itself exits with code 2, and in GitHub Actions a PowerShell step already fails as soon as that command returns nonzero Refs #2483
1 parent 97ec20b commit 570fab1

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

.github/workflows/lazarus.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,17 +50,16 @@ jobs:
5050
if: ${{ matrix.operating-system == 'windows-latest' }}
5151
shell: pwsh
5252
run: |
53-
.\extra\peflags\peflags.exe .\out\win64\heidisql.exe --check --policy
53+
& .\extra\peflags\peflags.exe .\out\win64\heidisql.exe --check --policy
5454
$code = $LASTEXITCODE
5555
Write-Host "peflags policy exit code: $code"
56-
5756
if (($code -band 1) -ne 0) { throw 'DYNAMICBASE not set' }
5857
if (($code -band 4) -ne 0) { throw 'NXCOMPAT not set' }
5958
if (($code -band 8) -ne 0) { throw 'HIGH_ENTROPY_VA not set on PE32+' }
60-
6159
if (($code -band 2) -ne 0) {
6260
Write-Warning 'ASLR is not effective on the Lazarus Windows build due to missing/unusable relocation directory.'
6361
}
62+
exit 0
6463
6564
- name: Upload binaries Windows
6665
if: ${{ matrix.operating-system == 'windows-latest' && matrix.lazarus-versions == 'stable' }}

0 commit comments

Comments
 (0)