On 04/14/2016 08:33 AM, Andrea Bolognani wrote:
There are a bunch of issues in the way we handle man pages;
most importantly, installed man pages (like libvirtd.8) end
up having stuff like
SYSCONFDIR/libvirtd.conf
in the FILES section.
This series makes it so all man pages are generated through the
same steps:
1. process $command.pod (shipped) with pod2man(1) to
convert pod markup to groff markup and obtain
$command.$section.in (shipped)
2. process $command.$section.in with sed(1) to insert
host-specific information such as @sysconfdir@
and obtain $command.$section (not shipped)
As a follow-up improvement, it would be great if we could
abstract those two steps into macros to reduce code duplication
even further. Please provide hints on the best way to
achieve that :)
A number of other minor issues are fixed along the way.
Cheers.
Andrea Bolognani (13):
build: Ship virt-admin.pod
build: Extract pod from source files
build: Build man pages in $(builddir)
build: Perform post-processing on all man pages
build: Standardize on .pod -> .x.in -> .x
build: Never ship man pages
build: Group files
build: Ensure intermediate files are cleaned up properly
build: Replace variables in man pages
build: Always ship virt-login-shell.conf
man: Fix NAME section
man: Fix SYNOPSIS section
man: Fix links
daemon/Makefile.am | 43 +++++----
daemon/{libvirtd.pod.in => libvirtd.pod} | 4 +-
src/Makefile.am | 69 ++++++++------
src/locking/{virtlockd.pod.in => virtlockd.pod} | 4 +-
src/logging/{virtlogd.pod.in => virtlogd.pod} | 4 +-
tools/Makefile.am | 114 +++++++++++++++---------
tools/virsh.pod | 6 +-
tools/virt-admin.pod | 6 +-
tools/virt-host-validate.c | 74 ---------------
tools/virt-host-validate.pod | 68 ++++++++++++++
tools/virt-login-shell.pod | 3 +-
tools/virt-pki-validate.in | 64 -------------
tools/virt-pki-validate.pod | 61 +++++++++++++
tools/virt-sanlock-cleanup.in | 52 -----------
tools/virt-sanlock-cleanup.pod | 49 ++++++++++
tools/virt-xml-validate.in | 113 -----------------------
tools/virt-xml-validate.pod | 111 +++++++++++++++++++++++
17 files changed, 440 insertions(+), 405 deletions(-)
rename daemon/{libvirtd.pod.in => libvirtd.pod} (97%)
rename src/locking/{virtlockd.pod.in => virtlockd.pod} (97%)
rename src/logging/{virtlogd.pod.in => virtlogd.pod} (97%)
create mode 100644 tools/virt-host-validate.pod
create mode 100644 tools/virt-pki-validate.pod
create mode 100644 tools/virt-sanlock-cleanup.pod
create mode 100644 tools/virt-xml-validate.pod
Changes seem reasonable to me (although not exactly my specialty).
Should I assume you ran make dist or distcheck to ensure everything gets
to the right place? I'm missing some things so those don't work for me
any more (first error for me was "recipe for target
'libvirt-daemon-arch.png' failed" - I didn't bother to dig).
ACK
John