[dpdk-dev] [PATCH 20.08 9/9] config/arm: support python3 only

Thomas Monjalon thomas at monjalon.net
Sat May 23 23:27:42 CEST 2020


22/05/2020 16:10, Kilheeney, Louise:
> From: Thomas Monjalon <thomas at monjalon.net> 
> > 22/05/2020 15:23, Louise Kilheeney:
> > > Changed script to explicitly use python3 only.
> > 
> > What is the reason of this change?
> 
> since python 2 is EOL, Making these scripts to use python3-only,
> it's part of a general update to have everything use python3.
> This is already using python 3 and this makes it explicit.

This is to avoid maintaining python 2 compatibility I guess.
Please insert this real reason in the git commit log.


> > --- a/config/arm/armv8_machine.py
> > +++ b/config/arm/armv8_machine.py
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/python
> > +#!/usr/bin/python3

Are you sure all OS have /usr/bin/python3?
Is it called "python" sometimes?
Is it always located in /usr/bin/?
Shouldn't we use "#! /usr/bin/env python3" ?




More information about the dev mailing list