Re: [PATCH 2/3] tests: remove "bc" from package listing

[Philippe Mathieu-Daudé]

On 11/4/20 2:19 PM, Daniel P. Berrangé wrote:
> There is no sign of 'bc' being used by any tests in the tree.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  scripts/coverity-scan/coverity-scan.docker         | 1 -
>  tests/docker/dockerfiles/debian10.docker           | 1 -
>  tests/docker/dockerfiles/fedora-win32-cross.docker | 1 -
>  tests/docker/dockerfiles/fedora-win64-cross.docker | 1 -
>  tests/docker/dockerfiles/fedora.docker             | 1 -
>  5 files changed, 5 deletions(-)

I added it to be able to rebuild easily the blobs consumed
by our tests:

commit 1bbf2d010fc0e5f0a9c5e1967368c44bc64e086a
Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
Date:   Mon Mar 16 17:21:28 2020 +0000

    tests/docker: Install tools to cross-debug and build Linux kernels

    We often run Linux kernels to test QEMU. We sometimes need
    to build them manually to use non-default features. We only
    miss the tiny 'bc' tool.

    The ncurses library is helpful to run 'make menuconfig'.

    Finally, gdb-multiarch allow us to debug a TCG guest when its
    architecture is different than the host.

I think this is related to the question of what we can include
to the repository for testing... If we can not add binary and
don't have how to produce them, it becomes complicated.