Skip to content

Commit

Permalink
Fix build to not use asan on relase mode (#308)
Browse files Browse the repository at this point in the history
- Not use ASAN for release build
- Remove cmake settings(replaced with presets)
- Adjust GHA triggers

Signed-off-by: Renato Foot <[email protected]>
  • Loading branch information
Costallat authored Apr 12, 2022
1 parent a9cd5c7 commit 87edf62
Show file tree
Hide file tree
Showing 17 changed files with 21 additions and 204 deletions.
30 changes: 1 addition & 29 deletions .github/ISSUE_TEMPLATE/bug.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
name: Report a problem/bug
description: Any issues that you have found while using the server
title: "[Bug]: "
labels: ["Status: Pending Test", "Type: Bug"]
body:
- type: markdown
Expand All @@ -21,7 +20,6 @@ body:
validations:
required: true


- type: checkboxes
id: area
attributes:
Expand All @@ -39,38 +37,12 @@ body:
id: what-happened
attributes:
label: What happened?
description: Describe the problem/bug!
description: Describe your problem/bug and add screenshots, logs or anything that can help verifying the bug
placeholder: Tell us what you see!
value: "A bug happened!"
validations:
required: true

- type: textarea
id: steps-reproduce
attributes:
label: How to Reproduce?
description: Steps to reproduce the problem, position on the map, etc
placeholder: Tell us exactly how to reproduce and the position on the map!
value: |
1 - go to position 134,543,7
2 - Try to open the door
3 - Door doesn't open!
validations:
required: true

- type: textarea
id: logs
attributes:
label: Relevant log output
description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backtick.
render: shell

- type: textarea
id: attach
attributes:
label: Attach any relevant file
description: Attach screenshots, full logs, etc ... that helps verifying the bug

- type: dropdown
id: os
attributes:
Expand Down
5 changes: 0 additions & 5 deletions .github/ISSUE_TEMPLATE/config.yml

This file was deleted.

29 changes: 1 addition & 28 deletions .github/ISSUE_TEMPLATE/content.yaml
Original file line number Diff line number Diff line change
@@ -1,26 +1,12 @@
name: Report a missing content
description: Any thing that is missing
title: "[Content]: "
labels: ["Status: Pending Test", "Type: Missing Content"]
body:
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this issue!
- type: dropdown
id: priority
attributes:
label: Priority
description: How critical is this?
options:
- Low
- Medium
- High
- Critical
validations:
required: true

- type: checkboxes
id: area
attributes:
Expand All @@ -38,23 +24,10 @@ body:
id: missing-content
attributes:
label: What is missing?
description: Map, sprite, script, quest, teleport, monster, etc
description: Map, sprite, script, quest, teleport, monster, etc. Attach screenshots, logs, etc, anything that can helps us
validations:
required: true

- type: textarea
id: logs
attributes:
label: Relevant log output
description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backtick.
render: shell

- type: textarea
id: attach
attributes:
label: Attach any relevant file
description: Attach screenshots, logs, etc, anything that can helps us

- type: checkboxes
id: terms
attributes:
Expand Down
22 changes: 1 addition & 21 deletions .github/ISSUE_TEMPLATE/enhancement.yaml
Original file line number Diff line number Diff line change
@@ -1,26 +1,12 @@
name: Report a enhancement
description: Any thing that is missing
title: "[Enhancement]: "
labels: ["Status: Pending Test", "Type: Enhancement"]
body:
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this issue!
- type: dropdown
id: priority
attributes:
label: Priority
description: How critical is this?
options:
- Low
- Medium
- High
- Critical
validations:
required: true

- type: checkboxes
id: area
attributes:
Expand All @@ -38,16 +24,10 @@ body:
id: enhancement
attributes:
label: Describe the enhancement
description: What should be improved
description: What should be improved? Attach screenshots, logs, etc, anything that can helps us
validations:
required: true

- type: textarea
id: attach
attributes:
label: Attach any relevant file
description: Attach screenshots, logs, etc, anything that can helps us

- type: checkboxes
id: terms
attributes:
Expand Down
10 changes: 0 additions & 10 deletions .github/ISSUE_TEMPLATE/i-want-to-help.md

This file was deleted.

32 changes: 0 additions & 32 deletions .github/ISSUE_TEMPLATE/map-issues.md

This file was deleted.

27 changes: 0 additions & 27 deletions .github/ISSUE_TEMPLATE/missing-feature.md

This file was deleted.

24 changes: 0 additions & 24 deletions .github/ISSUE_TEMPLATE/missing-wrong-content.md

This file was deleted.

3 changes: 0 additions & 3 deletions .github/workflows/analysis-codacy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@ on:
push:
branches:
- main
- dev
- devel
- develop

jobs:
codacy:
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/analysis-reviewdog.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
---
name: Analysis - Review Dog
on: [pull_request]

on:
pull_request:

jobs:

cppcheck:
Expand Down
3 changes: 0 additions & 3 deletions .github/workflows/analysis-sonarcloud.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@ on:
push:
branches:
- main
- dev
- devel
- develop

env:
VCPKG_BUILD_TYPE: release
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/build-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,11 @@
name: Build - Docker

on:
workflow_dispatch:
pull_request:
push:
branches:
- main
- dev
- devel
- develop

env:
VCPKG_BUILD_TYPE: release
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/build-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ name: Build - Ubuntu
on:
workflow_dispatch:
pull_request:
push:
branches:
- main

env:
VCPKG_BUILD_TYPE: release
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/build-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ name: Build - Windows
on:
workflow_dispatch:
pull_request:
push:
branches:
- main

env:
VCPKG_BUILD_TYPE: release
Expand Down
8 changes: 6 additions & 2 deletions CMakePresets.json
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,9 @@
"displayName": "Windows - Debug Build",
"description": "Build Debug Mode",
"cacheVariables": {
"CMAKE_BUILD_TYPE": "Debug"
"CMAKE_BUILD_TYPE": "Debug",
"DEBUG_LOG": "ON",
"ASAN_ENABLED": "ON"
},
"architecture": {
"value": "x64",
Expand All @@ -88,7 +90,9 @@
"displayName": "Linux - Debug Build",
"description": "Build Debug Mode",
"cacheVariables": {
"CMAKE_BUILD_TYPE": "Debug"
"CMAKE_BUILD_TYPE": "Debug",
"DEBUG_LOG": "ON",
"ASAN_ENABLED": "ON"
}
}
],
Expand Down
15 changes: 0 additions & 15 deletions CMakeSettings.json

This file was deleted.

2 changes: 1 addition & 1 deletion src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
# *****************************************************************************
option(OPTIONS_ENABLE_OPENMP "Enable Open Multi-Processing support." ON)
option(DEBUG_LOG "Enable Debug Log" OFF)
option(ASAN_ENABLED "Build this target with AddressSanitizer" ON)
option(ASAN_ENABLED "Build this target with AddressSanitizer" OFF)


# *****************************************************************************
Expand Down

0 comments on commit 87edf62

Please sign in to comment.