On Fri, Sep 27, 2019 at 12:33:05PM +0530, Numan Siddique wrote: > On Thu, Sep 26, 2019 at 11:12 PM Ben Pfaff <b...@ovn.org> wrote: > > > Python 2 reaches end-of-life on January 1, 2020, which is only > > a few months away. This means that OVS needs to stop depending > > on in the next release that should occur roughly that same time. > > Therefore, this commit removes all support for Python 2. It > > also makes Python 3 a mandatory build dependency. > > > > Some of the interesting consequences: > > > > - HAVE_PYTHON, HAVE_PYTHON2, and HAVE_PYTHON3 conditionals have > > been removed, since we now know that Python3 is available. > > > > - $PYTHON and $PYTHON2 are removed, and $PYTHON3 is always > > available. > > > > - Many tests for Python 2 support have been removed, and the ones > > that depended on Python 3 now run unconditionally. This allowed > > several macros in the testsuite to be removed, making the code > > clearer. This does make some of the changes to the testsuite > > files large due to indentation level changes. > > > > - #! lines for Python now use /usr/bin/python3 instead of > > /usr/bin/python. > > > > - Packaging depends on Python 3 packages. > > > > Hi Ben, > > The patch failed to apply on latest master, but it applied on top of the > commit > - 15a5dddde50a("travis: Drop -MD related workaround for sparse.") so I > tested > it on top of it. > > I tested this patch on Fedora 30 and centos 7 container. The compilation > and make check is successful. > > I also tested with "make rpm-fedora". > > Acked-by: Numan Siddique <nusid...@redhat.com> > Tested-by: Numan Siddique <nusid...@redhat.com> > > Few comments on "make rpm-fedora" on centos 7. > The build fails with the error python-six not found. After I passed > "RPMBUILD_OPT="--with build_python3" it's successful. > > May be we can enable python3 in openvswitch-fedora.spec.in by default since > there will > be no python2 support. But I think that can be a separate patch. > > Without this patch, "make rpm-fedora" on centos7 is successful but > python3-openvswitch package > is not generated.
Thanks. I applied this to master. Do you want to write up that additional patch? _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev