[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from th
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports |
Date: |
Tue, 5 Feb 2019 14:42:46 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi Peter,
On 2/5/19 2:20 PM, Peter Maydell wrote:
> On Tue, 29 Jan 2019 at 17:57, Philippe Mathieu-Daudé <address@hidden> wrote:
>>
>> Various iotests scripts (run via 'make check-block') use bash
>> specific extentions. OpenBSD comes with the Korn shell as default.
>> Install bash to be able to run those tests.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> tests/vm/openbsd | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/tests/vm/openbsd b/tests/vm/openbsd
>> index 6263c8956b..e9c2a3f2c8 100755
>> --- a/tests/vm/openbsd
>> +++ b/tests/vm/openbsd
>> @@ -45,6 +45,8 @@ class OpenBSDVM(basevm.BaseVM):
>> self.wait_ssh()
>> sys.stderr.write("Disabling W^X on the build partition...\n")
>> self.ssh_root_check("sed -E -i 's_(/tmp\ ffs)\ ([^\ ]*)_\\1
>> \\2,wxallowed_' /etc/fstab")
>> + sys.stderr.write("Installing bash...\n")
>> +
>> self.ssh_root_check("PKG_PATH=https://ftp.openbsd.org/pub/OpenBSD/6.1/packages/amd64
>> pkg_add bash")
>> self.ssh_root("shutdown -p now")
>> self.wait()
>>
>
> Wouldn't it make more sense to just update the image to include
> the necessary package, the same way we do with all QEMU's
> other build dependencies ?
Instead of updating the image, Daniel asked if we could upgrade this
image to a more recent release (to remove SDL1), but IIRC the outcome
was there is no manpower for that. Meanwhile, this kludge seems the
simplest way.
BTW This command is run once at image creation.
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports, Peter Maydell, 2019/02/05
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports,
Philippe Mathieu-Daudé <=
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports, Brad Smith, 2019/02/06
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports, Alex Bennée, 2019/02/06
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports, Brad Smith, 2019/02/06
- Re: [Qemu-devel] [PATCH v2 09/18] tests/vm/openbsd: Install Bash from the ports, Alex Bennée, 2019/02/07