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

[Peter Maydell]

On 19 June 2018 at 13:05, Eduardo Habkost <address@hidden> wrote:
> On Tue, Jun 19, 2018 at 12:13:41PM +0100, Peter Maydell wrote:
>> 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 <address@hidden>
> 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 <address@hidden>
>     Reviewed-by: Richard Henderson <address@hidden>
>
> tests/docker/docker.py is still python2-only but ./configure is
> trying to run it using $python (which can be Python 3).

Aha, yes -- I'd forgotten, but this config in my buildtest setup
is indeed configuring with --python=python3.

thanks
-- PMM