22/05/2020 16:10, Kilheeney, Louise: > From: Thomas Monjalon <tho...@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" ?