Page MenuHomePhabricator

[cmake] Add man pages for the new executables
ClosedPublic

Authored by Fabien on Wed, Nov 19, 09:01.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCb391ea877c5f: [cmake] Add man pages for the new executables
Summary

Both iguana and proof-manager-cli are lacking a target to build their man pages. Let's fix this so it can be added to our documentation on the website.

Test Plan
cmake -GNinja .. -DBUILD_CHRONIK=ON -DBUILD_CHRONIK_PLUGINS=ON -DBUILD_PROOF_MANAGER_CLI=ON -DCMAKE_INSTALL_PREFIX=install -DBUILD_IGUANA=ON
ninja install-manpages-html

Diff Detail

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

Event Timeline

Fabien requested review of this revision.Wed, Nov 19, 09:01
Fabien updated this revision to Diff 56669.

Rebase

PiRK added a subscriber: PiRK.
PiRK added inline comments.
src/iguana/CMakeLists.txt
36 ↗(On Diff #56669)

no need for the include on this one?

This revision is now accepted and ready to land.Wed, Nov 19, 10:14
src/iguana/CMakeLists.txt
36 ↗(On Diff #56669)

No because it's a subdir or src/ and the src/CMakeLists.txt already includes the file the defined values are available already