Page MenuHomePhabricator
Differential D4697

[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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
teamcity added a comment.Dec 11 2019, 07:49

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Dec 11 2019, 08:02

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8499: Diff 14764.Dec 11 2019, 08:02
teamcity added a comment.Dec 11 2019, 18:52

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
teamcity added a comment.Dec 11 2019, 19:10

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

PathSize
src/
8 lines

Diff 14793

View Options

src/CMakeLists.txt

Loading...