Page MenuHomePhabricator

[CMAKE] Silent the man pages generation
ClosedPublic

Authored by Fabien on Sep 17 2020, 07:02.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCf571f9774071: [CMAKE] Silent the man pages generation
Summary

This diff makes the man pages generation silent, unless there is a
failure.

Depends on D7473.

Test Plan
ninja install-manpages

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien requested review of this revision.Sep 17 2020, 07:02
deadalnix requested changes to this revision.Sep 17 2020, 13:02
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
cmake/modules/InstallationHelper.cmake
184 ↗(On Diff #23546)

None of this belongs here.

191 ↗(On Diff #23546)

dito

Especially this one. This is a good indicator that once you've put things in the wrong place, you get more and more of it over time.

This revision now requires changes to proceed.Sep 17 2020, 13:02
Fabien planned changes to this revision.Sep 18 2020, 07:47
Fabien edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Sep 18 2020, 23:00
This revision was automatically updated to reflect the committed changes.