[CMAKE] Add an option to promote some warnings to errors
ClosedPublic
Actions

Authored by Fabien on Dec 11 2019, 07:49.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGING8e85917cc541: [CMAKE] Add an option to promote some warnings to errors
rABC8e85917cc541: [CMAKE] Add an option to promote some warnings to errors

Summary

This is a cmake port of the autotools --enable-werror option.

Test Plan

cmake -GNinja -DENABLE_WERROR=ON ..
ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cmake_werror

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8499
Build 15002: Default Diff Build & Tests
Build 15001: arc lint + arc unit

Event Timeline

Fabien created this revision.Dec 11 2019, 07:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 11 2019, 07:49

Herald added a reviewer: deadalnix. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8499: Diff 14764.Dec 11 2019, 08:02

Build Bitcoin-ABC / Diffs / Diff Testing started.

jasonbcox accepted this revision.Dec 11 2019, 19:01

This revision is now accepted and ready to land.Dec 11 2019, 19:01

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Closed by commit rABC8e85917cc541: [CMAKE] Add an option to promote some warnings to errors (authored by Fabien). · Explain WhyDec 12 2019, 07:13

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC8e85917cc541: [CMAKE] Add an option to promote some warnings to errors.

Fabien added a commit: rSTAGING8e85917cc541: [CMAKE] Add an option to promote some warnings to errors.Apr 7 2020, 22:03

Revision Contents
Changeset List

Path

Size

src/

CMakeLists.txt

8 lines

Diff 14764

View Options

[CMAKE] Add an option to promote some warnings to errorsClosedPublicActions