mbox series

[0/3] Some cleanups to build deps

Message ID 20201104131924.593522-1-berrange@redhat.com
Headers show
Series Some cleanups to build deps | expand

Message

Daniel P. Berrangé Nov. 4, 2020, 1:19 p.m. UTC
In looking at using libvirt-ci to generate the docker files I noticed
some wierd build deps in current package lists. This cleans them up.

Daniel P. Berrangé (3):
  hw/usb/ccid: remove references to NSS
  tests: remove "bc" from package listing
  docker: remove libblockdev-mpath-devel package

 .travis.yml                                       | 14 +++++++-------
 docs/ccid.txt                                     | 15 +++++++--------
 scripts/coverity-scan/coverity-scan.docker        |  3 ---
 tests/docker/dockerfiles/centos7.docker           |  1 +
 tests/docker/dockerfiles/centos8.docker           |  1 +
 tests/docker/dockerfiles/debian10.docker          |  1 -
 .../docker/dockerfiles/fedora-win32-cross.docker  |  1 -
 .../docker/dockerfiles/fedora-win64-cross.docker  |  1 -
 tests/docker/dockerfiles/fedora.docker            |  4 +---
 9 files changed, 17 insertions(+), 24 deletions(-)

-- 
2.28.0

Comments

Philippe Mathieu-Daudé Nov. 4, 2020, 1:42 p.m. UTC | #1
On 11/4/20 2:19 PM, Daniel P. Berrangé wrote:
> This package was added in

> 

>   commit 8ea5962f286272ac2533892c07f9b7e2a77d8819

>   Author: Philippe Mathieu-Daudé <philmd@redhat.com>

>   Date:   Sat May 4 07:54:40 2019 +0200

> 

>     tests/docker: Test more components on the Fedora default image

> 

> but this is never used by QEMU. The multipath support in QEMU

> requires the device-mapper-multipath-devel package instead, which

> is already present.

> 

> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>

> ---

>  scripts/coverity-scan/coverity-scan.docker | 1 -

>  tests/docker/dockerfiles/fedora.docker     | 1 -

>  2 files changed, 2 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Philippe Mathieu-Daudé Nov. 4, 2020, 1:47 p.m. UTC | #2
On 11/4/20 2:19 PM, Daniel P. Berrangé wrote:
> The NSS package was previously pre-requisite for building CCID related

> features, however, this became obsolete when the libcacard library was

> spun off to a separate project:

> 

>     commit 7b02f5447c64d1854468f758398c9f6fe9e5721f

>     Author: Marc-André Lureau <marcandre.lureau@redhat.com>

>     Date:   Sun Aug 30 11:48:40 2015 +0200

> 

>         libcacard: use the standalone project

> 

> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>

> ---

>  .travis.yml                                | 14 +++++++-------

>  docs/ccid.txt                              | 15 +++++++--------

>  scripts/coverity-scan/coverity-scan.docker |  1 -

>  tests/docker/dockerfiles/centos7.docker    |  1 +

>  tests/docker/dockerfiles/centos8.docker    |  1 +

>  tests/docker/dockerfiles/fedora.docker     |  2 +-

>  6 files changed, 17 insertions(+), 17 deletions(-)

...

> diff --git a/docs/ccid.txt b/docs/ccid.txt

> index c7fda6d07d..1baf03e29b 100644

> --- a/docs/ccid.txt

> +++ b/docs/ccid.txt

> @@ -34,15 +34,14 @@ reader and smart card (i.e. not backed by a physical device) using this device.

>  

>  2. Building

>  

> -The cryptographic functions and access to the physical card is done via NSS.

> -

> -Installing NSS:

> +The cryptographic functions and access to the physical card is done via the

> +libcacard library, whose development package must be installed prior to

> +building QEMU:

>  

>  In redhat/fedora:

> -    yum install nss-devel

> -In ubuntu/debian:

> -    apt-get install libnss3-dev

> -    (not tested on ubuntu)

> +    yum install libcacard-devel

> +In ubuntu:

> +    apt-get install libcacard-dev

>  

>  Configuring and building:

>      ./configure --enable-smartcard && make

> @@ -51,7 +50,7 @@ Configuring and building:

>  3. Using ccid-card-emulated with hardware

>  

>  Assuming you have a working smartcard on the host with the current

> -user, using NSS, qemu acts as another NSS client using ccid-card-emulated:

> +user, using libcacard, qemu acts as another client using ccid-card-emulated:


Maybe replace 'qemu' by 'QEMU' (another instance line 125).

Otherwise:

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>


Thanks.