diff mbox series

[ovs-dev] debian: Add python3-sphinx to ovs build dependencies

Message ID 20200515191704.25555-1-aatteka@ovn.org
State New
Headers show
Series [ovs-dev] debian: Add python3-sphinx to ovs build dependencies | expand

Commit Message

Ansis Atteka May 15, 2020, 7:17 p.m. UTC
python3-sphinx has become mandatory build dependency since patch
39b5e46 ("Documentation: Convert multiple manpages to ReST."), because,
otherwise, packaging of OVS debian packages fails with an error that
generated man pages can't be foundl.

Fixes: 39b5e46312 ("Documentation: Convert multiple manpages to ReST.")
CC: Ben Pfaff <blp@ovn.org>
Signed-off-by: Ansis Atteka <aatteka@ovn.org>
Reported-by: Artem Teleshev <artem.teleshev@gmail.com>
---
 debian/control | 1 +
 1 file changed, 1 insertion(+)

Comments

Gregory Rose May 15, 2020, 8:32 p.m. UTC | #1
On 5/15/2020 12:17 PM, Ansis Atteka wrote:
> python3-sphinx has become mandatory build dependency since patch
> 39b5e46 ("Documentation: Convert multiple manpages to ReST."), because,
> otherwise, packaging of OVS debian packages fails with an error that
> generated man pages can't be foundl.
> 
> Fixes: 39b5e46312 ("Documentation: Convert multiple manpages to ReST.")
> CC: Ben Pfaff <blp@ovn.org>
> Signed-off-by: Ansis Atteka <aatteka@ovn.org>
> Reported-by: Artem Teleshev <artem.teleshev@gmail.com>
> ---
>   debian/control | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/debian/control b/debian/control
> index e47767d75..0646b22a1 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -14,6 +14,7 @@ Build-Depends: graphviz,
>                  openssl,
>                  procps,
>                  python3-all,
> +               python3-sphinx,
>                  python3-twisted,
>                  python3-zope.interface,
>                  libunbound-dev,
> 

Acked-by: Greg Rose <gvrose8192@gmail.com>
Ansis May 15, 2020, 10:03 p.m. UTC | #2
On Fri, May 15, 2020 at 1:32 PM Gregory Rose <gvrose8192@gmail.com> wrote:
>
>
> On 5/15/2020 12:17 PM, Ansis Atteka wrote:
> > python3-sphinx has become mandatory build dependency since patch
> > 39b5e46 ("Documentation: Convert multiple manpages to ReST."), because,
> > otherwise, packaging of OVS debian packages fails with an error that
> > generated man pages can't be foundl.
> >
> > Fixes: 39b5e46312 ("Documentation: Convert multiple manpages to ReST.")
> > CC: Ben Pfaff <blp@ovn.org>
> > Signed-off-by: Ansis Atteka <aatteka@ovn.org>
> > Reported-by: Artem Teleshev <artem.teleshev@gmail.com>
> > ---
> >   debian/control | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/debian/control b/debian/control
> > index e47767d75..0646b22a1 100644
> > --- a/debian/control
> > +++ b/debian/control
> > @@ -14,6 +14,7 @@ Build-Depends: graphviz,
> >                  openssl,
> >                  procps,
> >                  python3-all,
> > +               python3-sphinx,
> >                  python3-twisted,
> >                  python3-zope.interface,
> >                  libunbound-dev,
> >
>
> Acked-by: Greg Rose <gvrose8192@gmail.com>
Thanks pushed to master.
diff mbox series

Patch

diff --git a/debian/control b/debian/control
index e47767d75..0646b22a1 100644
--- a/debian/control
+++ b/debian/control
@@ -14,6 +14,7 @@  Build-Depends: graphviz,
                openssl,
                procps,
                python3-all,
+               python3-sphinx,
                python3-twisted,
                python3-zope.interface,
                libunbound-dev,