[dpdk-dev] [PATCH v9 00/11] adding support for python 3 only

[Kevin Laatz]

On 29/09/2020 13:22, David Marchand wrote:
> Hello Kevin,
>
> On Tue, Sep 29, 2020 at 12:27 PM Kevin Laatz <kevin.laatz at intel.com> wrote:
>> This patch set converts all python scripts in the project to use
>> python3 only and removes all deprecation notices associated with these
>> changes. This is due to python2 being EOL in January 2020.
>>
>> ---
>> v6:
>>    - rebased, removing conflict with make removal patchset.
>>    - added changes to buildtools/map_to_win.py
>>
>> v7:
>>    - typo in email Cc'ing David Marchand
>>    - added maintainers for buildtools patch
>>
>> v8:
>>    - removed unrelated cleanup
>>    - replaced integer cast with integer division operator
>>
>> v9:
>>    - include documentation updates
>>
>> Kevin Laatz (4):
>>    app/test-cmdline: support python3 only
>>    app/test: support python3 only
>>    buildtools: support python3 only
>>    doc: support python3 only
>>
>> Louise Kilheeney (7):
>>    usertools/dpdk-telemetry-client: support python3 only
>>    usertools/dpdk-devbind: support python3 only
>>    usertools/dpdk-pmdinfo: support python3 only
>>    usertools/cpu_layout: support python3 only
>>    devtools: support python3 only
>>    config/arm: support python3 only
>>    app/test-bbdev: support python3 only

Hi David,

> I can still see one issue in doc/guides/conf.py as you mentioned yesterday.
> There is also a script that directly calls /usr/bin/python3, worth
> fixing from my pov.
>
> $ for file in $(find app/ buildtools/ config/ devtools/ doc/ drivers/
> examples/ kernel/ lib/ license/ usertools/ -name "*.py"); do head -1
> $file |grep -q env.*python3 || echo $file; done
> doc/guides/conf.py
> usertools/dpdk-telemetry.py

doc/guides/conf.py is indirectly called from sphinx which potentially causes some compatibilty issues when we make conf.py only support python3 (if sphinx in using python2) - this is why I left this out of the patchset, otherwise the doc build will be broken :-(

I can change direct call to /usr/bin/python3.

-Kevin