Skip to content

Commit 6bdce2e

Browse files
committed
Remove -Werror=all from build flags, but enable it in CI
Fixes: #7024
1 parent 1847723 commit 6bdce2e

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

Diff for: .github/scripts/sketch_utils.sh

+1
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,7 @@ function build_sketch(){ # build_sketch <ide_path> <user_path> <path-to-ino> [ex
161161
--fqbn "$currfqbn" \
162162
--board-options "$curroptions" \
163163
--warnings "all" \
164+
--build-property "compiler.warning_flags.all=-Wall -Werror=all -Wextra" \
164165
--build-cache-path "$ARDUINO_CACHE_DIR" \
165166
--build-path "$build_dir" \
166167
$xtra_opts "${sketchdir}"

Diff for: platform.txt

+2-2
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ compiler.optimization_flags.debug=-Og -g3
4141
compiler.warning_flags=-w
4242
compiler.warning_flags.none=-w
4343
compiler.warning_flags.default=
44-
compiler.warning_flags.more=-Wall -Werror=all
45-
compiler.warning_flags.all=-Wall -Werror=all -Wextra
44+
compiler.warning_flags.more=-Wall
45+
compiler.warning_flags.all=-Wall -Wextra
4646

4747
# Compile Flags
4848
compiler.cpreprocessor.flags="@{compiler.sdk.path}/flags/defines" "-I{build.source.path}" -iprefix "{compiler.sdk.path}/include/" "@{compiler.sdk.path}/flags/includes" "-I{compiler.sdk.path}/{build.memory_type}/include"

0 commit comments

Comments
 (0)