Message ID | 20200928104328.409055-1-kevin.laatz@intel.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1E428A04C3; Mon, 28 Sep 2020 12:48:35 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 277101D71A; Mon, 28 Sep 2020 12:48:03 +0200 (CEST) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id CF49A1D6F8 for <dev@dpdk.org>; Mon, 28 Sep 2020 12:47:57 +0200 (CEST) IronPort-SDR: yBK050VbpTwEMa1x0YvGyapFEbhHfo4lQVNQTGutQ+WD/XlFDe3AweMiJppxvgxyUGo6hxtX5M zgnmuK1Jkbsg== X-IronPort-AV: E=McAfee;i="6000,8403,9757"; a="141988889" X-IronPort-AV: E=Sophos;i="5.77,313,1596524400"; d="scan'208";a="141988889" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Sep 2020 03:47:53 -0700 IronPort-SDR: e4dEqySPjLZhZyT3K746PVsFQo9ntqZ7csr4TVIg2sljLeP9G4fAO9pEgi8/WChXIJXxKHBDOn FBCNRK1UTGeQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,313,1596524400"; d="scan'208";a="488524269" Received: from silpixa00399838.ir.intel.com ([10.237.213.224]) by orsmga005.jf.intel.com with ESMTP; 28 Sep 2020 03:47:50 -0700 From: Kevin Laatz <kevin.laatz@intel.com> To: dev@dpdk.org Cc: bruce.richardson@intel.com, anatoly.burakov@intel.com, robin.jarry@6wind.com, david.marchand@redhat.com, Kevin Laatz <kevin.laatz@intel.com> Date: Mon, 28 Sep 2020 11:43:18 +0100 Message-Id: <20200928104328.409055-1-kevin.laatz@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200921120357.220588-1-kevin.laatz@intel.com> References: <20200921120357.220588-1-kevin.laatz@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v8 00/10] adding support for python 3 only X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
adding support for python 3 only
|
|
Message
Kevin Laatz
Sept. 28, 2020, 10:43 a.m. UTC
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 Kevin Laatz (3): app/test-cmdline: support python3 only app/test: support python3 only buildtools: 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 app/test-bbdev/test-bbdev.py | 7 +------ app/test-cmdline/cmdline_test.py | 9 ++------- app/test-cmdline/cmdline_test_data.py | 1 + app/test/autotest.py | 7 +------ app/test/autotest_data.py | 1 + app/test/autotest_runner.py | 21 ++++++++------------- app/test/autotest_test_funcs.py | 1 + buildtools/map_to_win.py | 3 +-- config/arm/armv8_machine.py | 2 +- devtools/update_version_map_abi.py | 7 +------ usertools/cpu_layout.py | 13 ++----------- usertools/dpdk-devbind.py | 22 ++++------------------ usertools/dpdk-pmdinfo.py | 7 +------ usertools/dpdk-telemetry-client.py | 18 +++--------------- 14 files changed, 28 insertions(+), 91 deletions(-)
Comments
Hi Kevin, all, 2020-09-28, Kevin Laatz: > 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 For all patches in the series: Acked-by: Robin Jarry <robin.jarry@6wind.com>
On Mon, Sep 28, 2020 at 01:29:53PM +0200, Robin Jarry wrote: > Hi Kevin, all, > > 2020-09-28, Kevin Laatz: > > 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 > > For all patches in the series: > > Acked-by: Robin Jarry <robin.jarry@6wind.com> Great. Looking forward to having this merged as I'm tired of seeing the warnings from our scripts when I forget to explicitly call them using python3! I expect, once applied, the deprecation notice should also be removed.
Hello Kevin, Louise, On Mon, Sep 28, 2020 at 12:48 PM Kevin Laatz <kevin.laatz@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. Thanks for working on this. I noticed some leftovers: - doc/guides/contributing/coding_style.rst and doc/guides/linux_gsg/sys_reqs.rst still mention that scripts work with python2, - the deprecation notice can be dropped and this will go with a corresponding update of the release note,
On Mon, Sep 28, 2020 at 05:09:30PM +0200, David Marchand wrote: > Hello Kevin, Louise, > > On Mon, Sep 28, 2020 at 12:48 PM Kevin Laatz <kevin.laatz@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. > > Thanks for working on this. > > I noticed some leftovers: > - doc/guides/contributing/coding_style.rst and > doc/guides/linux_gsg/sys_reqs.rst still mention that scripts work with > python2, For the GSG doc, there is an overlap with the make removal doc updates which already fix that (since meson itself requires py3): http://patches.dpdk.org/patch/78191/ Good catch on the former. > - the deprecation notice can be dropped and this will go with a > corresponding update of the release note, > > > -- > David Marchand >
On 28/09/2020 16:14, Bruce Richardson wrote: > On Mon, Sep 28, 2020 at 05:09:30PM +0200, David Marchand wrote: >> Hello Kevin, Louise, >> >> On Mon, Sep 28, 2020 at 12:48 PM Kevin Laatz <kevin.laatz@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. >> Thanks for working on this. >> >> I noticed some leftovers: >> - doc/guides/contributing/coding_style.rst and >> doc/guides/linux_gsg/sys_reqs.rst still mention that scripts work with >> python2, > For the GSG doc, there is an overlap with the make removal doc updates > which already fix that (since meson itself requires py3): > > http://patches.dpdk.org/patch/78191/ > > Good catch on the former. Just noticed doc/guides/conf.py was also missed... Will work on this and the above for v9 >> - the deprecation notice can be dropped and this will go with a >> corresponding update of the release note, Will take care of this in the v9 as well. Thanks. >> -- >> David Marchand >>