[Buildroot] [PATCH v2 3/4] utils/getdeveloperlib.py: use python3 explicitly

Petr Vorel petr.vorel at gmail.com
Thu Sep 16 19:51:42 UTC 2021


Hi Arnout,

> On 15/09/2021 21:48, Petr Vorel wrote:
> > + remove python2 compatibility imports

> > library is used by support/scripts/pkg-stats and utils/get-developers
> > which both use python3. Although library does not require shebang,
> > it's better to state python3 use.

>  Do you have some supporting evidence of this claim "it's better to add a shebang"?
No.

>  Looking at a few random projects with python files, I see none that do it.
I'm sorry, I was obviously wrong.

>  And it seems like a lot of noise to add this for no obvious gain. If it's about
> documenting the minimum python version, it's not even correct: some of these
> scripts will require python3.4 or python3.5 and that's not documented...
Agree => setting this commit and the 4th commit in patchwork as Rejected.

Thanks a lot for your input.

Kind regards,
Petr

>  Regards,
>  Arnout


More information about the buildroot mailing list