[ovs-dev] [PATCH] ovs-sim: Don't install manpage at all (except from ovs-sim itself).

[Ben Pfaff]

ovs-sim is a funny utility since it only works from a build tree, not from
an installed OVS.  That means that we shouldn't install its manpage when
we run "make install".  But we do want to install the manpage when we're
inside ovs-sim itself, so that the user can invoke "man ovs-sim" from its
nested shell.

This commit makes this happen.

Suggested-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
---
 Documentation/automake.mk       | 12 ++++++++----
 rhel/openvswitch-fedora.spec.in |  1 -
 rhel/openvswitch.spec.in        |  1 -
 utilities/ovs-sim.in            |  2 +-
 4 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/Documentation/automake.mk b/Documentation/automake.mk
index 2202df45b6cc..bc728dff3157 100644
--- a/Documentation/automake.mk
+++ b/Documentation/automake.mk
@@ -110,7 +110,7 @@ DOC_SOURCE = \
 	Documentation/internals/contributing/libopenvswitch-abi.rst \
 	Documentation/internals/contributing/submitting-patches.rst \
 	Documentation/requirements.txt \
-	$(addprefix Documentation/ref/,$(RST_MANPAGES))
+	$(addprefix Documentation/ref/,$(RST_MANPAGES) $(RST_MANPAGES_NOINST))
 FLAKE8_PYFILES += Documentation/conf.py
 EXTRA_DIST += $(DOC_SOURCE)
 
@@ -158,8 +158,12 @@ RST_MANPAGES = \
 	ovs-vlan-test.8.rst \
 	ovsdb-server.7.rst \
 	ovsdb.5.rst \
-	ovsdb.7.rst \
-	ovs-sim.1.rst
+	ovsdb.7.rst
+
+# rST formatted manpages that we don't want to install because they
+# document stuff that only works with a build tree, not with an
+# installed OVS.
+RST_MANPAGES_NOINST = ovs-sim.1.rst
 
 # The GNU standards say that these variables should control
 # installation directories for manpages in each section.  Automake
@@ -202,7 +206,7 @@ INSTALL_DATA_LOCAL += install-man-rst
 if HAVE_SPHINX
 install-man-rst: docs-check
 	@$(set_mandirs); \
-	for rst in $(RST_MANPAGES); do \
+	for rst in $(RST_MANPAGES) $(EXTRA_RST_MANPAGES); do \
 	    $(extract_stem_and_section); \
 	    echo " $(MKDIR_P) '$(DESTDIR)'\"$$mandir\""; \
 	    $(MKDIR_P) '$(DESTDIR)'"$$mandir"; \
diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index 9be16ffe7a31..22d0235f7131 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -320,7 +320,6 @@ install -p -D -m 0755 \
 # remove unpackaged files
 rm -f $RPM_BUILD_ROOT%{_bindir}/ovs-parse-backtrace \
         $RPM_BUILD_ROOT%{_sbindir}/ovs-vlan-bug-workaround \
-        $RPM_BUILD_ROOT%{_mandir}/man1/ovs-sim.1 \
         $RPM_BUILD_ROOT%{_mandir}/man8/ovs-vlan-bug-workaround.8
 
 %check
diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
index e8be048aa27f..2c5f0409a078 100644
--- a/rhel/openvswitch.spec.in
+++ b/rhel/openvswitch.spec.in
@@ -100,7 +100,6 @@ rm \
     $RPM_BUILD_ROOT/usr/share/man/man8/ovs-test.8 \
     $RPM_BUILD_ROOT/usr/share/man/man8/ovs-l3ping.8 \
     $RPM_BUILD_ROOT/usr/sbin/ovs-vlan-bug-workaround \
-    $RPM_BUILD_ROOT/usr/share/man/man1/ovs-sim.1 \
     $RPM_BUILD_ROOT/usr/share/man/man8/ovs-vlan-bug-workaround.8 \
     $RPM_BUILD_ROOT/usr/bin/ovn-* \
     $RPM_BUILD_ROOT/usr/share/man/man?/ovn-* \
diff --git a/utilities/ovs-sim.in b/utilities/ovs-sim.in
index 72436c08941b..47329da212d4 100755
--- a/utilities/ovs-sim.in
+++ b/utilities/ovs-sim.in
@@ -466,7 +466,7 @@ export -f ovn_attach
 # Easy access to OVS manpages.
 mkdir $sim_base/man
 mandir=`cd $sim_base/man && pwd`
-(cd "$sim_builddir" && ${MAKE-make} install-man install-man-rst mandir=$mandir >/dev/null)
+(cd "$sim_builddir" && ${MAKE-make} install-man install-man-rst mandir=$mandir EXTRA_RST_MANPAGES=ovs-sim.1.rst >/dev/null)
 MANPATH=$mandir:; export MANPATH
 
 export scripts
-- 
2.16.1