DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support
@ 2020-05-13 11:02 Bruce Richardson
  2020-05-13 11:54 ` Burakov, Anatoly
  2020-08-28 14:15 ` [dpdk-dev] [PATCH v2] buildtools/map_to_win.py: " Bruce Richardson
  0 siblings, 2 replies; 5+ messages in thread
From: Bruce Richardson @ 2020-05-13 11:02 UTC (permalink / raw)
  To: dev; +Cc: Bruce Richardson

The build scripts called from meson all use python3 as meson itself depends
upon python3. Therefore there is no need for python2 compatibility in these
script files.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 buildtools/map_to_def.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/buildtools/map_to_def.py b/buildtools/map_to_def.py
index 6775b54a9..9edde7a67 100644
--- a/buildtools/map_to_def.py
+++ b/buildtools/map_to_def.py
@@ -1,8 +1,7 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2019 Intel Corporation
 
-from __future__ import print_function
 import sys
 from os.path import dirname, basename, join, exists
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support
  2020-05-13 11:02 [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support Bruce Richardson
@ 2020-05-13 11:54 ` Burakov, Anatoly
  2020-05-13 12:48   ` Bruce Richardson
  2020-08-28 14:15 ` [dpdk-dev] [PATCH v2] buildtools/map_to_win.py: " Bruce Richardson
  1 sibling, 1 reply; 5+ messages in thread
From: Burakov, Anatoly @ 2020-05-13 11:54 UTC (permalink / raw)
  To: Bruce Richardson, dev

On 13-May-20 12:02 PM, Bruce Richardson wrote:
> The build scripts called from meson all use python3 as meson itself depends
> upon python3. Therefore there is no need for python2 compatibility in these
> script files.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---

I think there was a reimplementation of check_output there that isn't 
necessary any more either. (and hasn't been for some time...)

Otherwise,

Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>

-- 
Thanks,
Anatoly

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support
  2020-05-13 11:54 ` Burakov, Anatoly
@ 2020-05-13 12:48   ` Bruce Richardson
  2020-05-14 13:15     ` Burakov, Anatoly
  0 siblings, 1 reply; 5+ messages in thread
From: Bruce Richardson @ 2020-05-13 12:48 UTC (permalink / raw)
  To: Burakov, Anatoly; +Cc: dev

On Wed, May 13, 2020 at 12:54:21PM +0100, Burakov, Anatoly wrote:
> On 13-May-20 12:02 PM, Bruce Richardson wrote:
> > The build scripts called from meson all use python3 as meson itself depends
> > upon python3. Therefore there is no need for python2 compatibility in these
> > script files.
> > 
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > ---
> 
> I think there was a reimplementation of check_output there that isn't
> necessary any more either. (and hasn't been for some time...)
>
Not sure what you are referring to here. Perhaps a different script file? 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support
  2020-05-13 12:48   ` Bruce Richardson
@ 2020-05-14 13:15     ` Burakov, Anatoly
  0 siblings, 0 replies; 5+ messages in thread
From: Burakov, Anatoly @ 2020-05-14 13:15 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: dev

On 13-May-20 1:48 PM, Bruce Richardson wrote:
> On Wed, May 13, 2020 at 12:54:21PM +0100, Burakov, Anatoly wrote:
>> On 13-May-20 12:02 PM, Bruce Richardson wrote:
>>> The build scripts called from meson all use python3 as meson itself depends
>>> upon python3. Therefore there is no need for python2 compatibility in these
>>> script files.
>>>
>>> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
>>> ---
>>
>> I think there was a reimplementation of check_output there that isn't
>> necessary any more either. (and hasn't been for some time...)
>>
> Not sure what you are referring to here. Perhaps a different script file?
> 

Oh, sorry, yes. I slightly misread the patch description :)

-- 
Thanks,
Anatoly

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [dpdk-dev] [PATCH v2] buildtools/map_to_win.py: drop python2 support
  2020-05-13 11:02 [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support Bruce Richardson
  2020-05-13 11:54 ` Burakov, Anatoly
@ 2020-08-28 14:15 ` Bruce Richardson
  1 sibling, 0 replies; 5+ messages in thread
From: Bruce Richardson @ 2020-08-28 14:15 UTC (permalink / raw)
  To: dev; +Cc: Bruce Richardson, Anatoly Burakov

The build scripts called from meson all use python3 as meson itself depends
upon python3. Therefore there is no need for python2 compatibility in these
script files, as well as which python2 support is being dropped generally
from DPDK.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
V2: rebased for 20.11, since this script is not covered by set 
    http://patches.dpdk.org/project/dpdk/list/?series=11739
---
 buildtools/map_to_win.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/buildtools/map_to_win.py b/buildtools/map_to_win.py
index 2990b58634..2a6cb88605 100644
--- a/buildtools/map_to_win.py
+++ b/buildtools/map_to_win.py
@@ -1,8 +1,7 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2019 Intel Corporation
 
-from __future__ import print_function
 import sys
 from os.path import dirname, basename, join, exists
 
-- 
2.25.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-08-28 14:15 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-13 11:02 [dpdk-dev] [PATCH 20.08] buildtools/map_to_def.py: drop python2 support Bruce Richardson
2020-05-13 11:54 ` Burakov, Anatoly
2020-05-13 12:48   ` Bruce Richardson
2020-05-14 13:15     ` Burakov, Anatoly
2020-08-28 14:15 ` [dpdk-dev] [PATCH v2] buildtools/map_to_win.py: " Bruce Richardson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).