[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 08/14] tests/vm: Added configuration file support
From: |
Robert Foley |
Subject: |
Re: [PATCH v1 08/14] tests/vm: Added configuration file support |
Date: |
Fri, 14 Feb 2020 13:00:03 -0500 |
On Fri, 14 Feb 2020 at 11:54, Alex Bennée <address@hidden> wrote:
> > from socket_thread import SocketThread
> > +import yaml
>
> So this throws my setup on my Gentoo SynQuacer. Is this meant to be in
> the standard library or is this a separate dependency?
>
This is a separate dependency. On Ubuntu the package is python3-yaml.
This brings up an interesting point.
If the yaml dependency is not there, should we expect to gracefully
handle this in the python script, and
simply not allow yaml files? I suppose we could error out if a yaml
file is supplied
and ask for the dependency to get installed.
Or is there something we should do earlier, maybe in the configure, to
warn or error about the missing
dependency?
Thanks & Regards,
-Rob
> --
> Alex Bennée
- Re: [PATCH v1 04/14] tests/vm: give wait_ssh() option to wait for root, (continued)
- [PATCH v1 07/14] tests/vm: Add configuration to basevm.py, Robert Foley, 2020/02/05
- [PATCH v1 08/14] tests/vm: Added configuration file support, Robert Foley, 2020/02/05
- [PATCH v1 10/14] tests/vm: Add ability to select QEMU from current build., Robert Foley, 2020/02/05
- [PATCH v1 09/14] tests/vm: add --boot-console switch, Robert Foley, 2020/02/05
- [PATCH v1 11/14] tests/vm: allow wait_ssh() to specify command, Robert Foley, 2020/02/05
- [PATCH v1 12/14] tests/vm: Added a new script for ubuntu.aarch64., Robert Foley, 2020/02/05
- [PATCH v1 14/14] tests/vm: change scripts to use self._config, Robert Foley, 2020/02/05
- [PATCH v1 13/14] tests/vm: Added a new script for centos.aarch64., Robert Foley, 2020/02/05
- Re: [PATCH v1 00/14] tests/vm: Add support for aarch64 VMs, Alex Bennée, 2020/02/07