Re: [Qemu-devel] [PULL 00/13] Python queue, 2018-06-11

[Eduardo Habkost <ehabkost@redhat.com>]

On Tue, Jun 19, 2018 at 12:13:41PM +0100, Peter Maydell wrote:
> On 11 June 2018 at 18:42, Eduardo Habkost <ehabkost@redhat.com> wrote:
> > The following changes since commit 0d2fa03dae4fbe185a082f361342b1e30aed4582:
> >
> >   Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20180608' into staging (2018-06-08 16:26:51 +0100)
> >
> > are available in the Git repository at:
> >
> >   git://github.com/ehabkost/qemu.git tags/python-next-pull-request
> >
> > for you to fetch changes up to c7883412440905b41dde7e70f4af782932e80e90:
> >
> >   python: Remove scripts/ordereddict.py (2018-06-08 16:40:54 -0300)
> >
> > ----------------------------------------------------------------
> > Python queue, 2018-06-11
> >
> > * Make code compatible with Python 3 using 'futurize --stage1'
> > * Require Python >= 2.7 and remove Python 2.6 compatibility
> >   modules
> >
> > ----------------------------------------------------------------
> 
> Hi; I noticed that running configure on my machine can now produce a
> python traceback from docker.py:
> 
> make: Entering directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg'
> config-host.mak is out-of-date, running configure
> Traceback (most recent call last):
>   File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py",
> line 29, in <module>
>     from StringIO import StringIO
> ImportError: No module named 'StringIO'
> Install prefix    /usr/local
> [etc]
> 
> I don't know if this is because of this change, but this is the
> most recent change involving python...

This seems to be introduced by:

commit 51a12b51fd3693a16eb73362713a98d467e15af7
Author: Alex Bennée <alex.bennee@linaro.org>
Date:   Wed Apr 4 14:24:39 2018 +0100

    configure: add test for docker availability

    This tests for a working docker installation without sudo and sets up
    config-host.mak accordingly. This will be useful from cross compiling
    things in the future.

    Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
    Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

tests/docker/docker.py is still python2-only but ./configure is
trying to run it using $python (which can be Python 3).

-- 
Eduardo