On Wed, 8 Nov 2017 12:18:36 -0800 Ben Pfaff <b...@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 <acon...@redhat.com> wrote: > > > > > Guru Shetty <g...@ovn.org> writes: > > > > > > > On 31 October 2017 at 13:06, Aaron Conole <acon...@redhat.com> wrote: > > > > > > > > Ben Pfaff <b...@ovn.org> writes: > > > > > > > > > On Tue, Oct 31, 2017 at 03:47:35PM -0400, Aaron Conole wrote: > > > > >> Ben Pfaff <b...@ovn.org> writes: > > > > >> > > > > >> > On Fri, Oct 20, 2017 at 12:39:10AM -0700, Gurucharan Shetty > > > > wrote: > > > > >> >> Signed-off-by: Gurucharan Shetty <g...@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@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 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
