Added build-werror config to error on build warnings
ClosedPublic
Actions

Authored by jasonbcox on Oct 23 2019, 17:37.

Details

Reviewers

deadalnix
Fabien

Group Reviewers

Restricted Project

Commits

rSTAGINGf5aa717c9c97: Added build-werror config to error on build warnings
rABCf5aa717c9c97: Added build-werror config to error on build warnings

Summary

Adding this build to TeamCity will give us a way to track when VLA and thread-safety-analysis issues show up.

Test Plan

git checkout master
ABC_BUILD_NAME=build-werror ./build-configurations.sh  # passes

git checkout ebf7876  # Commit on master with thread-safety-analysis warnings
ABC_BUILD_NAME=build-werror ./build-configurations.sh  # fails

Diff Detail

Repository

rABC Bitcoin ABC

Branch

build-werror

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 7851
Build 13722: Bitcoin ABC Buildbot (legacy)
Build 13721: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Oct 23 2019, 17:37

Herald added a reviewer: deadalnix. · View Herald TranscriptOct 23 2019, 17:37

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7851: Diff 13667.Oct 23 2019, 17:49

Fabien accepted this revision.Oct 23 2019, 20:52

This revision is now accepted and ready to land.Oct 23 2019, 20:52

Closed by commit rABCf5aa717c9c97: Added build-werror config to error on build warnings (authored by jasonbcox). · Explain WhyOct 23 2019, 21:39

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rABCf5aa717c9c97: Added build-werror config to error on build warnings.

jasonbcox added a commit: rSTAGINGf5aa717c9c97: Added build-werror config to error on build warnings.Apr 7 2020, 21:29

Revision Contents
Changeset List

Path

Size

contrib/

teamcity/

build-configurations.sh

5 lines

Diff 13667

View Options

contrib/teamcity/build-configurations.sh

Added build-werror config to error on build warningsClosedPublicActions