mbox series

[XEN,0/3] Default to python3

Message ID 20200120115053.1010739-1-anthony.perard@citrix.com (mailing list archive)
Headers show
Series Default to python3 | expand

Message

Anthony PERARD Jan. 20, 2020, 11:50 a.m. UTC
Patch series available in this git branch:
https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1

Hi,

I think it's time for Xen to build with python3 by default.

The main reason for that is that QEMU upstream don't build with python 2.x
anymore, and the python binary selected by Xen build system is the one used
when building qemu-xen. So now osstest failed to build QEMU upstream.

Also, python2 is EOL.

FYI, the hypervisor build system already select python3 by default, this change
the tools side.

Cheers,

Anthony PERARD (3):
  automation: Only build QEMU if Python >= 3.5
  automation: updating container to have python3-config binary
  tools: Default to python3

 automation/build/debian/stretch-i386.dockerfile  | 1 +
 automation/build/debian/stretch.dockerfile       | 1 +
 automation/build/debian/unstable-i386.dockerfile | 1 +
 automation/build/debian/unstable.dockerfile      | 1 +
 automation/build/fedora/29.dockerfile            | 1 +
 automation/build/suse/opensuse-leap.dockerfile   | 1 +
 automation/build/ubuntu/bionic.dockerfile        | 1 +
 automation/build/ubuntu/trusty.dockerfile        | 1 +
 automation/build/ubuntu/xenial.dockerfile        | 1 +
 automation/scripts/build                         | 4 ++--
 tools/configure.ac                               | 2 +-
 11 files changed, 12 insertions(+), 3 deletions(-)

Comments

Anthony PERARD Jan. 20, 2020, 11:52 a.m. UTC | #1
On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> Patch series available in this git branch:
> https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> 
> Hi,
> 
> I think it's time for Xen to build with python3 by default.
> 
> The main reason for that is that QEMU upstream don't build with python 2.x
> anymore, and the python binary selected by Xen build system is the one used
> when building qemu-xen. So now osstest failed to build QEMU upstream.
> 
> Also, python2 is EOL.
> 
> FYI, the hypervisor build system already select python3 by default, this change
> the tools side.

I forgot to say that there's a osstest patch as well:
[OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
Wei Liu Jan. 27, 2020, 12:30 p.m. UTC | #2
On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > Patch series available in this git branch:
> > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > 
> > Hi,
> > 
> > I think it's time for Xen to build with python3 by default.
> > 
> > The main reason for that is that QEMU upstream don't build with python 2.x
> > anymore, and the python binary selected by Xen build system is the one used
> > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > 
> > Also, python2 is EOL.
> > 
> > FYI, the hypervisor build system already select python3 by default, this change
> > the tools side.
> 
> I forgot to say that there's a osstest patch as well:
> [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev

AIUI I don't need to wait for that patch to be applied before applying
this series. Let me know if I'm wrong.

Wei.

> 
> -- 
> Anthony PERARD
Anthony PERARD Jan. 27, 2020, 12:36 p.m. UTC | #3
On Mon, Jan 27, 2020 at 12:30:21PM +0000, Wei Liu wrote:
> On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> > On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > > Patch series available in this git branch:
> > > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > > 
> > > Hi,
> > > 
> > > I think it's time for Xen to build with python3 by default.
> > > 
> > > The main reason for that is that QEMU upstream don't build with python 2.x
> > > anymore, and the python binary selected by Xen build system is the one used
> > > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > > 
> > > Also, python2 is EOL.
> > > 
> > > FYI, the hypervisor build system already select python3 by default, this change
> > > the tools side.
> > 
> > I forgot to say that there's a osstest patch as well:
> > [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
> 
> AIUI I don't need to wait for that patch to be applied before applying
> this series. Let me know if I'm wrong.

It just going to prevent a push :-). All build of staging will fail. So,
the osstest patch is needed before applying the patch 3/3.

Cheers,
Wei Liu Jan. 27, 2020, 2:40 p.m. UTC | #4
On Mon, Jan 27, 2020 at 12:36:23PM +0000, Anthony PERARD wrote:
> On Mon, Jan 27, 2020 at 12:30:21PM +0000, Wei Liu wrote:
> > On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> > > On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > > > Patch series available in this git branch:
> > > > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > > > 
> > > > Hi,
> > > > 
> > > > I think it's time for Xen to build with python3 by default.
> > > > 
> > > > The main reason for that is that QEMU upstream don't build with python 2.x
> > > > anymore, and the python binary selected by Xen build system is the one used
> > > > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > > > 
> > > > Also, python2 is EOL.
> > > > 
> > > > FYI, the hypervisor build system already select python3 by default, this change
> > > > the tools side.
> > > 
> > > I forgot to say that there's a osstest patch as well:
> > > [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
> > 
> > AIUI I don't need to wait for that patch to be applied before applying
> > this series. Let me know if I'm wrong.
> 
> It just going to prevent a push :-). All build of staging will fail. So,
> the osstest patch is needed before applying the patch 3/3.

Ack. I will push the first two patches first.

BTW, have you updated the docker images in Gitlab CI?

Wei.

> 
> Cheers,
> 
> -- 
> Anthony PERARD
Wei Liu Jan. 27, 2020, 2:44 p.m. UTC | #5
On Mon, Jan 27, 2020 at 02:40:40PM +0000, Wei Liu wrote:
> On Mon, Jan 27, 2020 at 12:36:23PM +0000, Anthony PERARD wrote:
> > On Mon, Jan 27, 2020 at 12:30:21PM +0000, Wei Liu wrote:
> > > On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> > > > On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > > > > Patch series available in this git branch:
> > > > > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > > > > 
> > > > > Hi,
> > > > > 
> > > > > I think it's time for Xen to build with python3 by default.
> > > > > 
> > > > > The main reason for that is that QEMU upstream don't build with python 2.x
> > > > > anymore, and the python binary selected by Xen build system is the one used
> > > > > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > > > > 
> > > > > Also, python2 is EOL.
> > > > > 
> > > > > FYI, the hypervisor build system already select python3 by default, this change
> > > > > the tools side.
> > > > 
> > > > I forgot to say that there's a osstest patch as well:
> > > > [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
> > > 
> > > AIUI I don't need to wait for that patch to be applied before applying
> > > this series. Let me know if I'm wrong.
> > 
> > It just going to prevent a push :-). All build of staging will fail. So,
> > the osstest patch is needed before applying the patch 3/3.
> 
> Ack. I will push the first two patches first.
> 
> BTW, have you updated the docker images in Gitlab CI?

Never mind. You said that in the commit message already.

Wei.
Anthony PERARD Feb. 11, 2020, 11:19 a.m. UTC | #6
On Mon, Jan 27, 2020 at 02:40:40PM +0000, Wei Liu wrote:
> On Mon, Jan 27, 2020 at 12:36:23PM +0000, Anthony PERARD wrote:
> > On Mon, Jan 27, 2020 at 12:30:21PM +0000, Wei Liu wrote:
> > > On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> > > > On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > > > > Patch series available in this git branch:
> > > > > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > > > > 
> > > > > Hi,
> > > > > 
> > > > > I think it's time for Xen to build with python3 by default.
> > > > > 
> > > > > The main reason for that is that QEMU upstream don't build with python 2.x
> > > > > anymore, and the python binary selected by Xen build system is the one used
> > > > > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > > > > 
> > > > > Also, python2 is EOL.
> > > > > 
> > > > > FYI, the hypervisor build system already select python3 by default, this change
> > > > > the tools side.
> > > > 
> > > > I forgot to say that there's a osstest patch as well:
> > > > [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
> > > 
> > > AIUI I don't need to wait for that patch to be applied before applying
> > > this series. Let me know if I'm wrong.
> > 
> > It just going to prevent a push :-). All build of staging will fail. So,
> > the osstest patch is needed before applying the patch 3/3.
> 
> Ack. I will push the first two patches first.

osstest should be ready, could you push that last patch?

Thanks,
Wei Liu Feb. 11, 2020, 11:25 a.m. UTC | #7
On Tue, Feb 11, 2020 at 11:19:02AM +0000, Anthony PERARD wrote:
> On Mon, Jan 27, 2020 at 02:40:40PM +0000, Wei Liu wrote:
> > On Mon, Jan 27, 2020 at 12:36:23PM +0000, Anthony PERARD wrote:
> > > On Mon, Jan 27, 2020 at 12:30:21PM +0000, Wei Liu wrote:
> > > > On Mon, Jan 20, 2020 at 11:52:17AM +0000, Anthony PERARD wrote:
> > > > > On Mon, Jan 20, 2020 at 11:50:50AM +0000, Anthony PERARD wrote:
> > > > > > Patch series available in this git branch:
> > > > > > https://xenbits.xen.org/git-http/people/aperard/xen-unstable.git br.python3-default-v1
> > > > > > 
> > > > > > Hi,
> > > > > > 
> > > > > > I think it's time for Xen to build with python3 by default.
> > > > > > 
> > > > > > The main reason for that is that QEMU upstream don't build with python 2.x
> > > > > > anymore, and the python binary selected by Xen build system is the one used
> > > > > > when building qemu-xen. So now osstest failed to build QEMU upstream.
> > > > > > 
> > > > > > Also, python2 is EOL.
> > > > > > 
> > > > > > FYI, the hypervisor build system already select python3 by default, this change
> > > > > > the tools side.
> > > > > 
> > > > > I forgot to say that there's a osstest patch as well:
> > > > > [OSSTEST PATCH] ts-xen-build-prep: Install python3-dev
> > > > 
> > > > AIUI I don't need to wait for that patch to be applied before applying
> > > > this series. Let me know if I'm wrong.
> > > 
> > > It just going to prevent a push :-). All build of staging will fail. So,
> > > the osstest patch is needed before applying the patch 3/3.
> > 
> > Ack. I will push the first two patches first.
> 
> osstest should be ready, could you push that last patch?

Pushed.

Wei.