[Openvpn-devel] build: Fix missing install of man page in certain environments

Message ID 20201029213259.1636-1-davids@openvpn.net
State New
Headers show
Series
  • [Openvpn-devel] build: Fix missing install of man page in certain environments
Related show

Commit Message

David Sommerseth Oct. 29, 2020, 9:32 p.m.
It turns out the logic for dist_man_MANS was incorrectly put inside the
HAVE_PYDOCUTILS block.  This results in the man page being installed
only if python-docutils is installed and available.

The solution is simple, move the dist_man_MANS part outside the
python-docutils block.  The openvpn.8 file is prebuilt in source
tarballs and will thus be available.

Reported-By: Philip Brown <philip@pbdigital.org>
Tested-By: Philip Brown <philip@pbdigital.org>
Signed-off-by: David Sommerseth <davids@openvpn.net>
---

Note: This may have a negative impact on hosts running 'make install'
(which also happens via 'make distcheck') when using the git tree or
otherwise not having a pre-built doc/openvpn.8 file available.
---
 doc/Makefile.am | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Patch

diff --git a/doc/Makefile.am b/doc/Makefile.am
index 340dd553..df2f54a3 100644
--- a/doc/Makefile.am
+++ b/doc/Makefile.am
@@ -67,10 +67,11 @@  dist_html_DATA = openvpn.8.html
 CLEANFILES = \
 	 openvpn.8 openvpn.8.html
 
+endif
+
 if WIN32
 else
 dist_man_MANS = openvpn.8
 endif
-endif
 
 dist-hook : openvpn.8 openvpn.8.html