Message ID | 20200513110249.26667-1-bruce.richardson@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Thomas Monjalon |
Headers | show |
Series | [20.08] buildtools/map_to_def.py: drop python2 support | expand |
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/iol-intel-Performance | success | Performance Testing PASS |
ci/iol-nxp-Performance | success | Performance Testing PASS |
ci/iol-mellanox-Performance | success | Performance Testing PASS |
ci/travis-robot | success | Travis build: passed |
ci/Intel-compilation | success | Compilation OK |
ci/iol-testing | fail | Testing issues |
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>
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?
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 :)
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
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(-)