Hi Johann,

I tried to build ledger with -DBUILD_WEB_DOCS=1 but I didn't specify
-DBUILD_DOCS.  This fails because man2html/groff are not found because
the find_program command for these programs is only run if BUILD_DOCS
is enabled.

I'm wondering if BUILD_WEB_DOCS should imply BUILD_DOCS, as in the
patch below.

What do you think?  Do you think a different solution is better?

Initially I was wondering if it makes sense to allow BUILD_WEB_DOCS
without BUILD_DOCS since people might just want to build the web docs
without the PDF, but texi2pdf/TeX is optional anyway, so there's
probably no point in allowing this.


diff --git a/doc/CMakeLists.txt b/doc/CMakeLists.txt
index 96c6d51..f90e3b6 100644
--- a/doc/CMakeLists.txt
+++ b/doc/CMakeLists.txt
@@ -27,6 +27,10 @@ endif()
 
 ########################################################################
 
+if (BUILD_WEB_DOCS)
+  set(BUILD_DOCS 1)
+endif()
+
 if (BUILD_DOCS)
   find_program(MAKEINFO makeinfo)
   find_program(TEXI2PDF texi2pdf)

-- 
Martin Michlmayr
http://www.cyrius.com/

-- 

--- 
You received this message because you are subscribed to the Google Groups 
"Ledger" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to