On 8/7/19 8:06 AM, Paolo Bonzini wrote:
> On 07/08/19 14:40, Alex Bennée wrote:
>>
>> Paolo Bonzini <pbonz...@redhat.com> writes:
>>
>>> The tests/tcg rely a lot on per-target informations from
>>> the QEMU makefiles, but most of the definitions in there
>>> aren't really relevant to TCG tests.
>>>
>>> This series is just a cleanup, but it could also be
>>> a useful start in making it possible to compile tests/tcg
>>> out of QEMU's tree, and/or making it a submodule, and/or
>>> unifying the system emulation tests with kvm-unit-tests.
>>
>> Hmm something is throwing off configure and making it use my login shell
>> instead of /bin/sh:
>>
>> libpmem support no
>> libudev yes
>> default devices yes
>> ~/lsrc/qemu.git/tests/tcg/configure.sh (line 63): 'case' builtin not
>> inside of switch block
>> case $arch in
>> ^
>> <W> fish: Error while reading file
>> /home/alex/lsrc/qemu.git/tests/tcg/configure.sh
>
> It's the ${SHELL} you found in patch 3. The disadvantage of relying on
> #! is that some people have bash in /usr/bin/bash rather than /bin/bash.
> But we already assume /bin/bash elsewhere so I can drop it.Rather, we use '#!/usr/bin/env bash' to find bash anywhere. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
