Differential D3787

build: split warnings out of CXXFLAGS
ClosedPublic
Actions

Authored by Fabien on Jul 31 2019, 15:54.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGING9e5735f71d2e: build: split warnings out of CXXFLAGS
rABC9e5735f71d2e: build: split warnings out of CXXFLAGS

Summary

CXXFLAGS should not be modified anyway. Also, this will enable us to
selectively disable warnings.

Backport of core PR13306
https://github.com/bitcoin/bitcoin/pull/13306/files

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jul 31 2019, 15:54

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 31 2019, 15:54

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 B7021: Diff 10551.Jul 31 2019, 16:05

deadalnix accepted this revision.Jul 31 2019, 18:21

This revision is now accepted and ready to land.Jul 31 2019, 18:21

Closed by commit rABC9e5735f71d2e: build: split warnings out of CXXFLAGS (authored by Cory Fields <cory-nospam-@coryfields.com>, committed by Fabien). · Explain WhyAug 1 2019, 09:59

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC9e5735f71d2e: build: split warnings out of CXXFLAGS.

Fabien added a commit: rSTAGING9e5735f71d2e: build: split warnings out of CXXFLAGS.Sep 19 2019, 01:32

Revision Contents
Changeset List

Path

Size

configure.ac

24 lines

src/

Makefile.am

2 lines

Diff 10563

View Options

configure.ac

View Options

build: split warnings out of CXXFLAGSClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 10563

configure.ac

src/Makefile.am

build: split warnings out of CXXFLAGS
ClosedPublic
Actions

Revision Contents
Changeset List