Opened 3 years ago

Closed 3 years ago

#1382 closed Bug / Defect (fixed)

openvpn(8) not installed unless python-docutils available

Reported by: _wiz_ Owned by:
Priority: minor Milestone: release 2.5.1
Component: Documentation Version: OpenVPN 2.5.0 (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc: tct, David Sommerseth

Description

The distribution file contains the openvpn(8) man page, but it's not installed unless python-docutils is installed. However, python-docutils would only be needed to generate the man page from the rst file, which is already a step that happens during the creating of the distribution, so the requirement is not necessary.

The attached patch fixes this.

Attachments (1)

patch-doc_Makefile.am (361 bytes) - added by _wiz_ 3 years ago.
Patch fixing the problem.

Download all attachments as: .zip

Change History (5)

Changed 3 years ago by _wiz_

Attachment: patch-doc_Makefile.am added

Patch fixing the problem.

comment:1 Changed 3 years ago by tct

Cc: tct added

comment:2 Changed 3 years ago by tct

Cc: David Sommerseth added

comment:3 Changed 3 years ago by Gert Döring

Milestone: release 2.5.1

A patch for this has been merged into release/2.5 already in October

commit bbac1542cfb4a9d3033999b26813f0dd0618c3f0
Author: David Sommerseth <davids@…>
Date: Thu Oct 29 22:32:59 2020 +0100

build: Fix missing install of man page in certain environments


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.

so this will be fixed in upcoming 2.5.1 (1-2 weeks ETA).

comment:4 Changed 3 years ago by Gert Döring

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.