[ovs-dev] [PATCH] [RPM] Update openvswitch-kmod-fedora.spec

2017-07-05 Thread Leif Madsen
e the RPM dependencies needed to build this RPM. Signed-off-by: Leif Madsen --- rhel/openvswitch-kmod-fedora.spec.in | 30 +- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/rhel/openvswitch-kmod-fedora.spec.in b/rhel/openvswitch-kmod-fedora.spec.in

Re: [ovs-dev] [PATCH] [RPM] Update openvswitch-kmod-fedora.spec

2017-07-13 Thread Leif Madsen
On Thu, Jul 13, 2017 at 10:20 AM, Aaron Conole wrote: > Hi Leif, > Ohai! > > Leif Madsen writes: > > > Update the kernel module spec file to better reflect how I'm > > locally building the kmod RPM in mock and pushing it to COPR. > > > > These ch

Re: [ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

2017-03-02 Thread Leif Madsen
Just an FYI that this patch breaks CentOS 7 builds because of the python3-devel dependency. Might need to wrap it in a conditional to check for CentOS 7 platform builds. CC: Russell and Flavio Thanks! Leif. On Tue, Feb 21, 2017 at 10:29 AM, Timothy Redaelli wrote: > Reported-at: https://bugzil

Re: [ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

2017-03-02 Thread Leif Madsen
On Thu, Mar 2, 2017 at 3:44 PM, Russell Bryant wrote: > > On Thu, Mar 2, 2017 at 3:42 PM, Leif Madsen wrote: > >> Just an FYI that this patch breaks CentOS 7 builds because of the >> python3-devel dependency. Might need to wrap it in a conditional to check >> f

[ovs-dev] [PATCH] packaging: Make Fedora spec file CentOS compatible

2017-03-02 Thread Leif Madsen
On CentOS, the package names aren't prefixed with python2, but rather are prefixed with simply python. This change addresses that and fixes up some documentation that was outdated, and updates the Vagrantfile to use the proper spec file and package names. Signed-off-by: Leif M

Re: [ovs-dev] [PATCH] packaging: Make Fedora spec file CentOS compatible

2017-03-02 Thread Leif Madsen
On Thu, Mar 2, 2017 at 10:04 PM, Leif Madsen wrote: > On CentOS, the package names aren't prefixed with python2, but rather > are prefixed with simply python. This change addresses that and fixes > up some documentation that was outdated, and updates the Vagrantfile > to use the

Re: [ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

2017-03-06 Thread Leif Madsen
me additional documentation that I've found. Leif. -- Leif Madsen | Partner Engineer - NFV & CI NFV Partner Engineering Red Hat GPG: (D670F846) BEE0 336E 5406 42BA 6194 6831 B38A 291E D670 F846 ___ dev mailing list d...@openvswitch.org https://

[ovs-dev] [PATCH v2] packaging: Make Fedora spec file CentOS compatible

2017-03-06 Thread Leif Madsen
On CentOS, the package names aren't prefixed with python2, but rather are prefixed with simply python. This change addresses that and fixes up some documentation that was outdated, and updates the Vagrantfile to use the proper spec file and package names. Signed-off-by: Leif M

[ovs-dev] [PATCH v3] packaging: Make Fedora spec file CentOS compatible

2017-03-06 Thread Leif Madsen
On CentOS, the package names aren't prefixed with python2, but rather are prefixed with simply python. This change addresses that and fixes up some documentation that was outdated, and updates the Vagrantfile to use the proper spec file and package names. Signed-off-by: Leif M

Re: [ovs-dev] [PATCH v3] packaging: Make Fedora spec file CentOS compatible

2017-03-06 Thread Leif Madsen
This patch v3 is ready for review. I've tested it on various distribution versions ( https://copr.fedorainfracloud.org/coprs/leifmadsen/ovs-master/build/522596/) and am satisfied I've accounted for all of the feedback previously received including: * reverting changes to Vagrantfile because Fedora

Re: [ovs-dev] [RFC 2/4] doc: Convert ovs-vlan-test to rST

2017-04-24 Thread Leif Madsen
roff versions of the manpages into $(DESTDIR)/usr/share/man/man#/ > (or wherever they are configured to go). > > (I have not actually tested either kind of packaging.) -- Leif Madsen | Partner Engineer - NFV & CI NFV Partner Engineering R

[ovs-dev] [PATCH] [packaging] Remove unneeded manpage globs

2017-04-24 Thread Leif Madsen
Remove unneeded manpage globs from the RPM spec file since they are no longer being generated. --- rhel/openvswitch-fedora.spec.in | 2 -- 1 file changed, 2 deletions(-) diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in index dbcab00..1095cd0 100644 --- a/rhel/openvsw

Re: [ovs-dev] [RFC 2/4] doc: Convert ovs-vlan-test to rST

2017-04-24 Thread Leif Madsen
On Mon, Apr 24, 2017 at 3:57 PM, Ben Pfaff wrote: > On Mon, Apr 24, 2017 at 03:52:53PM -0400, Leif Madsen wrote: > > I think this change might have broken packaging :) > > > > I just tested, and with the removal / renaming of the man8 pages for > > ovs-test and ovs-

Re: [ovs-dev] [RFC 2/4] doc: Convert ovs-vlan-test to rST

2017-04-26 Thread Leif Madsen
This is indeed the correct approach it seems. I've validated that adding Lance's change results in a successful build with the original globs. I'll ACK his patch here in a moment. Thanks! Leif. On Tue, Apr 25, 2017 at 4:03 PM, Lance Richardson wrote: > > From: "Ben Pf

Re: [ovs-dev] [RFC 2/4] doc: Convert ovs-vlan-test to rST

2017-04-26 Thread Leif Madsen
On Wed, Apr 26, 2017 at 10:33 AM, Ben Pfaff wrote: > On Tue, Apr 25, 2017 at 04:03:55PM -0400, Lance Richardson wrote: > > I ran into this a little while ago. The problem in my case was that > sphinx-build > > was not installed, so these man pages were not being built (seems odd > that this > > w

Re: [ovs-dev] [RFC 2/4] doc: Convert ovs-vlan-test to rST

2017-04-26 Thread Leif Madsen
On Wed, Apr 26, 2017 at 11:00 AM, Stephen Finucane wrote: > On Wed, 2017-04-26 at 10:55 -0400, Lance Richardson wrote: > > The rules to build rst-formatted man pages in the top-level > > Makefile.in are predicated on HAVE_SPHINX_TRUE, which does give the > > impression that Sphinx is optional. >

Re: [ovs-dev] [PATCH] rhel: add python-sphinx as a build dependency

2017-04-26 Thread Leif Madsen
On Tue, Apr 25, 2017 at 4:48 PM, Lance Richardson wrote: > The python-sphinx package is now required in order to build > man pages, add this package as a build requirement. > > Reported-by: Leif Madsen > Signed-off-by: Lance Richardson > --- > rhel/openvswitch-fedora.sp

Re: [ovs-dev] [PATCH] [packaging] Remove unneeded manpage globs

2017-04-26 Thread Leif Madsen
On Mon, Apr 24, 2017 at 4:07 PM, Ben Pfaff wrote: > On Mon, Apr 24, 2017 at 04:00:30PM -0400, Leif Madsen wrote: > > Remove unneeded manpage globs from the RPM spec file since they > > are no longer being generated. > > --- > > rhel/openvswitch-fedora.spec.in |