[ovs-dev] [PATCH ovn] docs: Clarify that OVS submodule should not be used to build OVS itself.
Ilya Maximets
i.maximets at ovn.org
Thu Mar 11 20:38:19 UTC 2021
It's unclear from the documentation that users should not use OVS from
the submodule to build actual OVS binaries (ovs-vswitchd or
ovsdb-server). Released and supported versions of Open vSwitch should
be used for that purpose.
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
---
Documentation/intro/install/general.rst | 29 ++++++++++++++++---------
1 file changed, 19 insertions(+), 10 deletions(-)
diff --git a/Documentation/intro/install/general.rst b/Documentation/intro/install/general.rst
index bbd303227..08a03a632 100644
--- a/Documentation/intro/install/general.rst
+++ b/Documentation/intro/install/general.rst
@@ -67,9 +67,18 @@ need the following software:
- Open vSwitch (https://docs.openvswitch.org/en/latest/intro/install/).
Open vSwitch is included as a submodule in the OVN source code. It is
- kept at the minimum recommended version for OVN to operate optimally.
- See below for instructions about how to use a different OVS source
- location.
+ kept at the minimum recommended version for OVN to build and operate
+ optimally. See below for instructions about how to use a different OVS
+ source location.
+
+ .. note::
+
+ These OVS sources used as a set of libraries to build OVN binaries, so
+ OVS submodule is only recommended to build OVN and *not recommended*
+ to be used as a source for OVS build. To actually build/run OVS binaries
+ (``ovs-vswitchd``, ``ovsdb-server``) use `released versions of
+ Open vSwitch <https://www.openvswitch.org/download/>`_ or packages
+ provided in your distribution.
- GNU make
@@ -157,8 +166,8 @@ the "configure" script::
$ ./boot.sh
-Before configuring OVN, build Open vSwitch. The easiest way to do this
-is to use the included OVS submodule in the OVN source::
+Before configuring OVN, prepare Open vSwitch sources. The easiest way to do
+this is to use the included OVS submodule in the OVN source tree::
$ git submodule update --init
$ cd ovs
@@ -167,11 +176,11 @@ is to use the included OVS submodule in the OVN source::
$ make
$ cd ..
-It is not required to use the included OVS submodule; however the OVS
-submodule is guaranteed to be the minimum recommended version of OVS
-to ensure OVN's optimal operation. If you wish to use OVS source code
-from a different location on the file system, then be sure to configure
-and build OVS before building OVN.
+It is not required to build with the included OVS submodule; however the OVS
+submodule is guaranteed to include minimum recommended version of OVS libraries
+to ensure OVN's build and optimal operation. If you wish to build with OVS
+source code from a different location on the file system, then be sure to
+configure and build it before building OVN.
.. _general-configuring:
--
2.26.2
More information about the dev
mailing list