Differential D8877

[CI] Work around LCOV stupid behavior
ClosedPublic
Actions

Authored by Fabien on Jan 11 2021, 20:36.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rABC66997ade0d56: [CI] Work around LCOV stupid behavior

Summary

D8874 was tested using a recent lcov version. Previous version are
stupid enough to print their output to stderr (when no error) instead of
stdout.
This diff works around this by redirecting stderr to stdout, so the
coverage-summary.txt file is not empty.

Test Plan

On Debian:

./contrib/teamcity/build-configurations.py cashtab-tests

Check the coverage summary has content in it.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

ci_cashtab_coverage_summary_lcov_bug

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 14710
Build 29379: Build Diff
Build 29378: arc lint + arc unit

Event Timeline

Fabien created this revision.Jan 11 2021, 20:36

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 11 2021, 20:36

Fabien requested review of this revision.Jan 11 2021, 20:36

Harbormaster completed remote builds in B14710: Diff 26864.Jan 11 2021, 20:36

deadalnix accepted this revision.Jan 11 2021, 23:11

This revision is now accepted and ready to land.Jan 11 2021, 23:11

Closed by commit rABC66997ade0d56: [CI] Work around LCOV stupid behavior (authored by Fabien). · Explain WhyJan 12 2021, 07:52

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC66997ade0d56: [CI] Work around LCOV stupid behavior.

Revision Contents
Changeset List

Path

Size

contrib/

teamcity/

build-configurations.yml

2 lines

Diff 26864

View Options

contrib/teamcity/build-configurations.yml

[CI] Work around LCOV stupid behaviorClosedPublicActions