[Chronik] Set Chronik C++ standard to C++17
ClosedPublic
Actions

Authored by tobias_ruck on Jul 26 2022, 19:11.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC7122a475a89b: [Chronik] Set Chronik C++ standard to C++17

Summary

The other C++ code requires C++17, and Chronik requires it, too.

Test Plan

ninja check-crates

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-cpp-standard

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19728
Build 39175: Build Diff	build-chronik
Build 39174: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Jul 26 2022, 19:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 26 2022, 19:11

tobias_ruck requested review of this revision.Jul 26 2022, 19:11

Harbormaster completed remote builds in B19728: Diff 34521.Jul 26 2022, 19:14

Fabien accepted this revision.Jul 27 2022, 10:13

This revision is now accepted and ready to land.Jul 27 2022, 10:13

Closed by commit rABC7122a475a89b: [Chronik] Set Chronik C++ standard to C++17 (authored by tobias_ruck). · Explain WhyJul 27 2022, 13:47

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC7122a475a89b: [Chronik] Set Chronik C++ standard to C++17.

Revision Contents
Changeset List

Path

Size

chronik/

CMakeLists.txt

3 lines

Diff 34521

View Options

[Chronik] Set Chronik C++ standard to C++17ClosedPublicActions