docs: meson: Change what 'docs' target builds
authorAndres Freund <[email protected]>
Tue, 21 Nov 2023 01:46:40 +0000 (17:46 -0800)
committerAndres Freund <[email protected]>
Tue, 21 Nov 2023 01:46:40 +0000 (17:46 -0800)
This undoes the change in what the 'docs' target builds 969509c3f2e. Tom was
concerned with having a target to just build the html docs, which a prior
commit now provided explicitly.

A subsequent commit will overhaul the documentation for the documentation
targets.

While at it, move all target in doc/src/sgml/Makefile up to just after the
default "html" target, and add a comment explaining "all" is *not* the default
target.

Reviewed-by: Peter Eisentraut <[email protected]>
Discussion: https://postgr.es/m/20230209203855[email protected]
Discussion: https://postgr.es/m/20231103163848[email protected]

doc/src/sgml/Makefile
doc/src/sgml/docguide.sgml
doc/src/sgml/meson.build

index 49d35dd0d6ece55624d4b27390f759acaf3665ea..53100828a6484264f7b78bf7c5005e67016b8c38 100644 (file)
@@ -15,6 +15,8 @@
 # Make "html" the default target, since that is what most people tend
 # to want to use.
 html:
+# Note that all is *not* the default target in this directory
+all: html man
 
 # We don't need the tree-wide headers or install support here.
 NO_GENERATED_HEADERS=yes
@@ -25,8 +27,6 @@ top_builddir = ../../..
 include $(top_builddir)/src/Makefile.global
 
 
-all: html man
-
 
 ifndef DBTOEPUB
 DBTOEPUB = $(missing) dbtoepub
index fd72b3975ad288dc028e269174e4b14749262181..b319621fdb867ec5235c5e858af975a89a9dfca6 100644 (file)
@@ -442,7 +442,7 @@ LOGLEVEL=-Dorg.apache.commons.logging.simplelog.defaultlog=WARN
    <para>
     To build just the <acronym>HTML</acronym> version of the documentation:
 <screen>
-<prompt>build$ </prompt><userinput>ninja docs</userinput>
+<prompt>build$ </prompt><userinput>ninja html</userinput>
 </screen>
     To build all forms of the documentation:
 <screen>
index e5a38e060c723258a3650e53bb3b30626bce3807..dfdb1d0daa7df4632ddba6c09c01b471025d0c00 100644 (file)
@@ -235,9 +235,9 @@ if docs_dep.found()
   alias_target('man', man)
   alias_target('install-man', install_doc_man)
 
-  # even though we don't want to build man pages as part of 'docs', we do want
-  # to install them as part of install-docs
+  # built and installed as part of the the docs target
   installdocs += install_doc_man
+  docs += man
 endif