Page MenuHomePhabricator
Differential D4147

[CMAKE] Install the man pages
ClosedPublic
Actions

Authored by Fabien on Sep 24 2019, 15:14.

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Project
Commits
rSTAGING6c467dfc4f6b: [CMAKE] Install the man pages
rABC6c467dfc4f6b: [CMAKE] Install the man pages
Summary

This adds support for the man pages installation.
The last directory is hardcoded to man1. This is correct as long as
the man files are hardcoded to section 1 only.

Test Plan

For all platforms:

mkdir -p buildcmake/install && cd install
cmake -GNinja .. -DCMAKE_INSTALL_PREFIX="install" # Other options

depending on the platform

ninja
ninja install

Check that the man pages are installed under install/share/man/man1.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmake_install_man
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7571
Build 13182: Bitcoin ABC Buildbot (legacy)
Build 13181: arc lint + arc unit

Event Timeline

Fabien created this revision.Sep 24 2019, 15:14
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 24 2019, 15:14
Herald added a reviewer: deadalnix. · View Herald Transcript
teamcity added a comment.Sep 24 2019, 15:14

Build Bitcoin-ABC / Diffs / Diff Testing started.

Harbormaster completed remote builds in B7571: Diff 13114.Sep 24 2019, 15:24
teamcity added a comment.Sep 24 2019, 15:24

Build Bitcoin-ABC / Diffs / Diff Testing passed.

jasonbcox requested changes to this revision.Sep 24 2019, 22:36
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
doc/man/CMakeLists.txt
11

s/CLI/TX/

This revision now requires changes to proceed.Sep 24 2019, 22:36
Fabien updated this revision to Diff 13132.Sep 25 2019, 05:46

Fix copy/paste error.

teamcity added a comment.Sep 25 2019, 05:46

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Sep 25 2019, 05:57

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7583: Diff 13132.Sep 25 2019, 05:57
jasonbcox accepted this revision.Sep 25 2019, 17:18
This revision is now accepted and ready to land.Sep 25 2019, 17:18
Closed by commit rABC6c467dfc4f6b: [CMAKE] Install the man pages (authored by Fabien). · Explain WhySep 25 2019, 18:49
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABC6c467dfc4f6b: [CMAKE] Install the man pages.
Fabien added a commit: rSTAGING6c467dfc4f6b: [CMAKE] Install the man pages.Sep 27 2019, 22:49

Revision Contents
Changeset List

PathSize
doc/
2 lines
man/
17 lines

Diff 13114

View Options

doc/CMakeLists.txt

Loading...
View Options

doc/man/CMakeLists.txt

Loading...