Soft Patch Panel
 help / color / mirror / Atom feed
* [spp] [PATCH 0/2] Instructions for building docs on Ubuntu 18.04
@ 2019-02-13 14:21 ogawa.yasufumi
  2019-02-13 14:21 ` [spp] [PATCH 1/2] docs: fix warning for invalid code examples ogawa.yasufumi
  2019-02-13 14:21 ` [spp] [PATCH 2/2] docs: add desc to build docs on Ubuntu 18.04 ogawa.yasufumi
  0 siblings, 2 replies; 3+ messages in thread
From: ogawa.yasufumi @ 2019-02-13 14:21 UTC (permalink / raw)
  To: spp, ferruh.yigit, ogawa.yasufumi

From: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>

This series of update is to add modification and instruction for
building docs on Ubuntu 18.04. Required package is changed and
validation for syntax of docs is more stricted on Ubuntu 18.04.

Yasufumi Ogawa (2):
  docs: fix warning for invalid code examples
  docs: add desc to build docs on Ubuntu 18.04

 docs/guides/api_ref/proc_independ.rst | 1 -
 docs/guides/api_ref/spp_mirror.rst    | 2 +-
 docs/guides/api_ref/spp_nfv.rst       | 2 +-
 docs/guides/api_ref/spp_primary.rst   | 8 +++++++-
 docs/guides/api_ref/spp_vf.rst        | 4 ++--
 docs/guides/setup/getting_started.rst | 7 +++++++
 6 files changed, 18 insertions(+), 6 deletions(-)

-- 
2.17.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [spp] [PATCH 1/2] docs: fix warning for invalid code examples
  2019-02-13 14:21 [spp] [PATCH 0/2] Instructions for building docs on Ubuntu 18.04 ogawa.yasufumi
@ 2019-02-13 14:21 ` ogawa.yasufumi
  2019-02-13 14:21 ` [spp] [PATCH 2/2] docs: add desc to build docs on Ubuntu 18.04 ogawa.yasufumi
  1 sibling, 0 replies; 3+ messages in thread
From: ogawa.yasufumi @ 2019-02-13 14:21 UTC (permalink / raw)
  To: spp, ferruh.yigit, ogawa.yasufumi

From: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>

For compiling docs with some version of sphinx-build, for example 1.8.4
on Ubuntu 18.04, code syntax in code-block should be valid. This example
is invalid and warned because it includes `....` even if it is just an
example.

    ..  code-block:: json

        [
            {
                "type": "primary"
            },
            ....
        ]

This patch is to correct syntax of code examples.

Signed-off-by: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>
---
 docs/guides/api_ref/proc_independ.rst | 1 -
 docs/guides/api_ref/spp_mirror.rst    | 2 +-
 docs/guides/api_ref/spp_nfv.rst       | 2 +-
 docs/guides/api_ref/spp_primary.rst   | 8 +++++++-
 docs/guides/api_ref/spp_vf.rst        | 4 ++--
 5 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/docs/guides/api_ref/proc_independ.rst b/docs/guides/api_ref/proc_independ.rst
index 2e73a1d..52e3002 100644
--- a/docs/guides/api_ref/proc_independ.rst
+++ b/docs/guides/api_ref/proc_independ.rst
@@ -62,5 +62,4 @@ Response example
         "type": "nfv",
         "client-id": 2
       }
-      ...
     ]
diff --git a/docs/guides/api_ref/spp_mirror.rst b/docs/guides/api_ref/spp_mirror.rst
index a19eac7..1d4efee 100644
--- a/docs/guides/api_ref/spp_mirror.rst
+++ b/docs/guides/api_ref/spp_mirror.rst
@@ -316,7 +316,7 @@ Detach tx port of ``ring:1`` from component named ``mr1``.
 .. code-block:: console
 
     $ curl -X PUT -H 'application/json' \
-      -d '{"action": "detach", "port": "ring:0", "dir": "tx"} \
+      -d '{"action": "detach", "port": "ring:0", "dir": "tx"}' \
       http://127.0.0.1:7777/v1/mirrors/1/components/mr1/ports
 
 
diff --git a/docs/guides/api_ref/spp_nfv.rst b/docs/guides/api_ref/spp_nfv.rst
index c30da06..949f772 100644
--- a/docs/guides/api_ref/spp_nfv.rst
+++ b/docs/guides/api_ref/spp_nfv.rst
@@ -85,7 +85,7 @@ Response example
       "client-id": 1,
       "status": "running",
       "ports": [
-        "phy:0", "phy:1", "vhost:0", "vhost:1", "ring:0", "ring:1", ...
+        "phy:0", "phy:1", "vhost:0", "vhost:1", "ring:0", "ring:1"
       ],
       "patches": [
         {
diff --git a/docs/guides/api_ref/spp_primary.rst b/docs/guides/api_ref/spp_primary.rst
index 3cbad10..276c7e9 100644
--- a/docs/guides/api_ref/spp_primary.rst
+++ b/docs/guides/api_ref/spp_primary.rst
@@ -125,7 +125,13 @@ Response example
           "tx": 0,
           "tx_drop": 0
         },
-        ...
+        {
+          "id": 2,
+          "rx": 0,
+          "rx_drop": 0,
+          "tx": 0,
+          "tx_drop": 0
+        }
       ]
     }
 
diff --git a/docs/guides/api_ref/spp_vf.rst b/docs/guides/api_ref/spp_vf.rst
index c406938..173adc4 100644
--- a/docs/guides/api_ref/spp_vf.rst
+++ b/docs/guides/api_ref/spp_vf.rst
@@ -139,7 +139,7 @@ Response example
     {
       "client-id": 1,
       "ports": [
-        "phy:0", "phy:1", "vhost:0", "vhost:1", "ring:0", "ring:1", ...
+        "phy:0", "phy:1", "vhost:0", "vhost:1", "ring:0", "ring:1"
       ],
       "components": [
         {
@@ -440,7 +440,7 @@ Request example
 .. code-block:: console
 
     $ curl -X PUT -H 'application/json' \
-      -d '{"action": "detach", "port": "vhost:0", "dir": "tx"} \
+      -d '{"action": "detach", "port": "vhost:0", "dir": "tx"}' \
       http://127.0.0.1:7777/v1/vfs/1/components/fwd1/ports
 
 Response
-- 
2.17.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [spp] [PATCH 2/2] docs: add desc to build docs on Ubuntu 18.04
  2019-02-13 14:21 [spp] [PATCH 0/2] Instructions for building docs on Ubuntu 18.04 ogawa.yasufumi
  2019-02-13 14:21 ` [spp] [PATCH 1/2] docs: fix warning for invalid code examples ogawa.yasufumi
@ 2019-02-13 14:21 ` ogawa.yasufumi
  1 sibling, 0 replies; 3+ messages in thread
From: ogawa.yasufumi @ 2019-02-13 14:21 UTC (permalink / raw)
  To: spp, ferruh.yigit, ogawa.yasufumi

From: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>

Instruction for building documentation is only for Ubuntu 16.04 LTS and
it is not considered for 18.04. It is required to install `latexmk` if
you use 18.04, and this update is to add the instruction.

Signed-off-by: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>
---
 docs/guides/setup/getting_started.rst | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/docs/guides/setup/getting_started.rst b/docs/guides/setup/getting_started.rst
index 9847903..8cda22b 100644
--- a/docs/guides/setup/getting_started.rst
+++ b/docs/guides/setup/getting_started.rst
@@ -387,6 +387,13 @@ For PDF, inkscape and latex packages are required.
     $ sudo apt install texlive-latex-extra
     $ sudo apt install texlive-latex-recommended
 
+You might also need to install ``latexmk`` in addition to if you use
+Ubuntu 18.04 LTS.
+
+.. code-block:: console
+
+    $ sudo apt install latexmk
+
 HTML documentation is compiled by running make with ``doc-html``. This
 command launch sphinx for compiling HTML documents.
 Compiled HTML files are created in ``docs/guides/_build/html/`` and
-- 
2.17.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-02-13 14:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-13 14:21 [spp] [PATCH 0/2] Instructions for building docs on Ubuntu 18.04 ogawa.yasufumi
2019-02-13 14:21 ` [spp] [PATCH 1/2] docs: fix warning for invalid code examples ogawa.yasufumi
2019-02-13 14:21 ` [spp] [PATCH 2/2] docs: add desc to build docs on Ubuntu 18.04 ogawa.yasufumi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).