Re: [PATCH 3/6] build: introduce rst2html as a mandatory build tool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 08, 2019 at 11:21:09AM +0000, Daniel P. Berrangé wrote:
The rst2html tool is provided by python docutils, and as the name
suggests, it converts RST documents into HTML.

This enables us to start writing docs on our website in RST format
instead of HTML, without changing the rest of our website templating
system away from XSLT yet.

Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
---
libvirt.spec.in              | 2 ++
m4/virt-external-programs.m4 | 5 +++++
mingw-libvirt.spec.in        | 1 +
3 files changed, 8 insertions(+)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index dcad08cb5f..84efe47ffc 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -260,6 +260,8 @@ BuildRequires: automake
BuildRequires: gettext-devel
BuildRequires: libtool
BuildRequires: /usr/bin/pod2man
+# Replace with python3-docutils when we drop py2 support
+BuildRequires: /usr/bin/rst2html
%endif
BuildRequires: gcc
BuildRequires: git
diff --git a/m4/virt-external-programs.m4 b/m4/virt-external-programs.m4
index 0f995998c3..b7779f159e 100644
--- a/m4/virt-external-programs.m4
+++ b/m4/virt-external-programs.m4
@@ -33,6 +33,11 @@ AC_DEFUN([LIBVIRT_CHECK_EXTERNAL_PROGRAMS], [
  then
    AC_MSG_ERROR("xsltproc is required to build libvirt")
  fi
+  AC_PATH_PROG([RST2HTML], [rst2html], [])

The program is called 'rst2html.py' on my Gentoo system
(from the dev-python/docutils package)

Jano

+  if test -z "$RST2HTML"
+  then
+    AC_MSG_ERROR("rst2html is required to build libvirt")
+  fi
  AC_PATH_PROG([AUGPARSE], [augparse], [/usr/bin/augparse])
  AC_PROG_MKDIR_P
  AC_PROG_LN_S

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux