[v2] crypto: fix build with gcrypt

[PATCH v2 0/2] crypto: fix build with gcrypt

Daniel P. Berrangé posted 2 patches 3 years, 8 months ago

Diff against v1
Download series mbox

Test docker-quick@centos7 passed

Test docker-mingw@fedora passed

Test checkpatch passed

Test FreeBSD failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200901133050.381844-1-berrange@redhat.com

.gitlab-ci.yml                          | 69 +++++++++++++++++++++++++
configure                               |  2 +
crypto/meson.build                      | 42 +++++++++++----
meson.build                             |  5 ++
tests/docker/dockerfiles/centos7.docker |  2 +
tests/docker/dockerfiles/centos8.docker |  1 +
6 files changed, 110 insertions(+), 11 deletions(-)

Expand all Fold all

[PATCH v2 0/2] crypto: fix build with gcrypt

Posted by Daniel P. Berrangé 3 years, 8 months ago

The build system failed to add gcrypt flags and also didn't link to
gnutls in all scenarios.  This was missed because of the lack of CI
coverage for various build scenarios

Changed in v2:

 - Change way we add library dependencies in meson rules
   to fix linux-user build with gcrypt/gnutls too.
 - Extend CI coverage to test 1 system and 1 linux-user
   build with each crypto combination, not merely tools.

Daniel P. Berrangé (2):
  crypto: fix build with gcrypt enabled
  gitlab: expand test coverage for crypto builds

 .gitlab-ci.yml                          | 69 +++++++++++++++++++++++++
 configure                               |  2 +
 crypto/meson.build                      | 42 +++++++++++----
 meson.build                             |  5 ++
 tests/docker/dockerfiles/centos7.docker |  2 +
 tests/docker/dockerfiles/centos8.docker |  1 +
 6 files changed, 110 insertions(+), 11 deletions(-)

-- 
2.26.2