Skip to content

Commit

Permalink
Add windows relelease build with address sanitizer (#326)
Browse files Browse the repository at this point in the history
Adds new release build with asan enabled to help tracking problems

Signed-off-by: Renato Foot renato@foot.br
  • Loading branch information
Costallat authored Apr 21, 2022
1 parent f81f904 commit 8552c10
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,6 @@ jobs:
- name: Create and Upload Artifact
uses: actions/upload-artifact@v3
with:
name: canary-${{ runner.os }}-${{ matrix.buildtype }}-${{ github.sha }}
name: canary-${{ matrix.buildtype }}-${{ github.sha }}
path: |
${{ github.workspace }}/build/${{ matrix.buildtype }}/bin/
4 changes: 2 additions & 2 deletions .github/workflows/build-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
strategy:
matrix:
os: [windows-2019, windows-2022]
buildtype: [windows-release, windows-debug]
buildtype: [windows-release, windows-release-asan, windows-debug]
include:
- os: windows-2019
triplet: x64-windows
Expand Down Expand Up @@ -60,6 +60,6 @@ jobs:
- name: Create and Upload Artifact
uses: actions/upload-artifact@v3
with:
name: canary-${{ runner.os }}-${{ matrix.buildtype }}-${{ github.sha }}
name: canary-${{ matrix.buildtype }}-${{ github.sha }}
path: |
${{ github.workspace }}/build/${{ matrix.buildtype }}/bin/
17 changes: 17 additions & 0 deletions CMakePresets.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,19 @@
"rhs": "Linux"
}
},
{
"name": "windows-release-asan",
"inherits": "windows-release",
"displayName": "Windows - Release + ASAN",
"description": "Release Mode with ASAN",
"cacheVariables": {
"ASAN_ENABLED": "ON"
},
"architecture": {
"value": "x64",
"strategy": "external"
}
},
{
"name": "windows-debug",
"inherits": "windows-release",
Expand Down Expand Up @@ -93,6 +106,10 @@
"name": "windows-release",
"configurePreset": "windows-release"
},
{
"name": "windows-release-asan",
"configurePreset": "windows-release-asan"
},
{
"name": "windows-debug",
"configurePreset": "windows-debug"
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

![GitHub repo size](https://img.shields.io/github/repo-size/opentibiabr/canary)

[![Codacy Badge](https://app.codacy.com/project/badge/Grade/8efdc281acbe4352bfb9706142da4b8e)](https://www.codacy.com/gh/opentibiabr/canary/dashboard?utm_source=github.com&utm_medium=referral&utm_content=opentibiabr/canary&utm_campaign=Badge_Grade)
[![Codacy Badge](https://app.codacy.com/project/badge/Grade/b4539fb9fc96444f8db450f62a537b20)](https://www.codacy.com/gh/opentibiabr/canary/dashboard?utm_source=github.com&utm_medium=referral&utm_content=opentibiabr/canary&utm_campaign=Badge_Grade)
[![Analysis - CodeQL](https://github.com/opentibiabr/canary/actions/workflows/analysis-codeql.yml/badge.svg)](https://github.com/opentibiabr/canary/actions/workflows/analysis-codeql.yml)
[![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=opentibiabr_canary&metric=alert_status)](https://sonarcloud.io/dashboard?id=opentibiabr_canary)

Expand Down

0 comments on commit 8552c10

Please sign in to comment.