[ovs-dev] [PATCH v1] ovs container build.sh requires python3

[Ginwala, Aliasgar]

Thanks Ben:

Yes ovs container build script is for ovs and similar for ovn; as user just using ovs in containers don't need to clone/use ovn stuff and vice versa.


Regards,
Aliasgar
________________________________
From: Ben Pfaff <blp at ovn.org>
Sent: Tuesday, October 15, 2019 9:17 AM
To: aginwala <aginwala at asu.edu>
Cc: aginwala <amginwal at gmail.com>; ovs dev <dev at openvswitch.org>; Ginwala, Aliasgar <aginwala at ebay.com>
Subject: Re: [ovs-dev] [PATCH v1] ovs container build.sh requires python3

Done.

Does it make sense for this to be in both repos?

On Mon, Oct 14, 2019 at 07:48:14PM -0700, aginwala wrote:
> Hi Ben:
>
> Can you also push this to OVS master? Got it merged in ovn repo already.
>
> On Fri, Oct 11, 2019 at 2:06 PM <amginwal at gmail.com> wrote:
>
> > From: Aliasgar Ginwala <aginwala at ebay.com>
> >
> > building ovn/ovs container breaks while configure:
> > checking for Python 3 (version 3.4 or later)... no
> > configure: error: Python 3.4 or later is required but not found in
> > /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin,
> > please install it or set  to point to it
> >
> > As per commit 1ca0323e7c29dc7ef5a615c265df0460208f92de
> > Require Python 3 and remove support for Python 2.
> >
> > Signed-off-by: Aliasgar Ginwala <aginwala at ebay.com>
> > ---
> >  utilities/docker/debian/build-kernel-modules.sh | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/utilities/docker/debian/build-kernel-modules.sh
> > b/utilities/docker/debian/build-kernel-modules.sh
> > index 1b12720b9..18ac35764 100755
> > --- a/utilities/docker/debian/build-kernel-modules.sh
> > +++ b/utilities/docker/debian/build-kernel-modules.sh
> > @@ -18,8 +18,8 @@ GITHUB_SRC=$3
> >
> >  # Install deps
> >  linux="linux-image-$KERNEL_VERSION linux-headers-$KERNEL_VERSION"
> > -build_deps="apt-utils libelf-dev build-essential libssl-dev python \
> > -python-six wget gdb autoconf libtool git automake bzip2 debhelper \
> > +build_deps="apt-utils libelf-dev build-essential libssl-dev python3 \
> > +python3-six wget gdb autoconf libtool git automake bzip2 debhelper \
> >  dh-autoreconf openssl"
> >
> >  apt-get update
> > --
> > 2.20.1 (Apple Git-117)
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.openvswitch.org%2Fmailman%2Flistinfo%2Fovs-dev&amp;data=02%7C01%7Caginwala%40ebay.com%7C62afd1ca221b4a7c5a6208d751961565%7C46326bff992841a0baca17c16c94ea99%7C0%7C0%7C637067577358976965&amp;sdata=rOj%2FaN1rFMeSgLacb55aa%2BmT4d2Tl3Ddphfb9q6Z3n8%3D&amp;reserved=0
> >
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.openvswitch.org%2Fmailman%2Flistinfo%2Fovs-dev&amp;data=02%7C01%7Caginwala%40ebay.com%7C62afd1ca221b4a7c5a6208d751961565%7C46326bff992841a0baca17c16c94ea99%7C0%7C0%7C637067577358976965&amp;sdata=rOj%2FaN1rFMeSgLacb55aa%2BmT4d2Tl3Ddphfb9q6Z3n8%3D&amp;reserved=0