[ovs-dev] [PATCH] rhel.rst: Add python-sphinx as a dependency.
Flavio Leitner
fbl at sysclose.org
Tue Nov 14 22:34:52 UTC 2017
On Wed, 8 Nov 2017 12:18:36 -0800
Ben Pfaff <blp at ovn.org> wrote:
> On Wed, Nov 08, 2017 at 03:31:48PM -0200, Flavio Leitner wrote:
> > On Wed, 01 Nov 2017 13:21:22 -0400
> > Aaron Conole <aconole at redhat.com> wrote:
> >
> > > Guru Shetty <guru at ovn.org> writes:
> > >
> > > > On 31 October 2017 at 13:06, Aaron Conole <aconole at redhat.com> wrote:
> > > >
> > > > Ben Pfaff <blp at ovn.org> writes:
> > > >
> > > > > On Tue, Oct 31, 2017 at 03:47:35PM -0400, Aaron Conole wrote:
> > > > >> Ben Pfaff <blp at ovn.org> writes:
> > > > >>
> > > > >> > On Fri, Oct 20, 2017 at 12:39:10AM -0700, Gurucharan Shetty wrote:
> > > > >> >> Signed-off-by: Gurucharan Shetty <guru at ovn.org>
> > > > >> >> ---
> > > > >> >> Documentation/intro/install/rhel.rst | 2 +-
> > > > >> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >> >>
> > > > >> >> diff --git a/Documentation/intro/install/rhel.rst b/Documentation/intro/install/rhel.rst
> > > > >> >> index 86c5cf3..aff6ccf 100644
> > > > >> >> --- a/Documentation/intro/install/rhel.rst
> > > > >> >> +++ b/Documentation/intro/install/rhel.rst
> > > > >> >> @@ -76,7 +76,7 @@ the below command::
> > > > >> >>
> > > > >> >> $ yum install gcc make python-devel openssl-devel kernel-devel graphviz \
> > > > >> >> kernel-debug-devel autoconf automake rpm-build redhat-rpm-config \
> > > > >> >> - libtool checkpolicy selinux-policy-devel
> > > > >> >> + libtool checkpolicy selinux-policy-devel python-sphinx
> > > > >> >
> > > > >> > For Debian, we just recommend installing the build-dependencies listed
> > > > >> > in debian/control. That has the advantage that it can't get out of
> > > > >> > date. It has the disadvantage, though, that it's not easy to cut and
> > > > >> > paste (although "apt-get build-dep openvswitch" usually does the trick).
> > > > >> > Maybe "yum" has some mode that installs dependencies from a spec file?
> > > > >>
> > > > >> For 'yum' distributions:
> > > > >>
> > > > >> yum-builddep
> > > > >>
> > > > >> For 'dnf' distributions (newer Fedora, and future RHEL versions):
> > > > >>
> > > > >> dnf builddep
> > > > >
> > > > > Would it be reasonable to change rhel.rst to recommend using one of
> > > > > those tools, to ease future maintenance?
> > > >
> > > > Sure. Something like below? I don't know about the wordsmithing, so
> > > > I'll defer that to Guru.
> > > >
> > > > ---
> > > >
> > > > diff --git a/Documentation/intro/install/rhel.rst b/Documentation/intro/install/
> > > > rhel.rst
> > > > index 86c5cf3..36bb661 100644
> > > > --- a/Documentation/intro/install/rhel.rst
> > > > +++ b/Documentation/intro/install/rhel.rst
> > > > @@ -72,11 +72,14 @@ Build Requirements
> > > >
> > > > To compile the RPMs, you will need to install the packages described in the
> > > > :doc:`general` along with some additional packages. These can be installed with
> > > > -the below command::
> > > > +the below command for ``yum`` based distributions (but note that the
> > > > +openvswitch source RPM must be available somewhere)::
> > > >
> > > > Expecting source rpm while building the rpm, looks like is not going to happen for majority of the
> > > > cases. I took it that we will need it if "yum-builddep openvswitch" needs to succeed?.
> > >
> > > I didn't think about. As I understand, `yum-builddep` requires the
> > > source RPM. That means we would need to build the SRPM (rpmbuild -bs)
> > > first.
> >
> > You can use both tools with the spec file:
> > [fleitner at plex ovs]$ sudo yum-builddep rhel/openvswitch-fedora.spec
>
> Hey, that seems about perfect. Can we recommend using that?
I posted this patch that uses in-tree spec file and also uses
RPM Development Tools group to simplify the process:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340929.html
Thanks,
--
Flavio
More information about the dev
mailing list