[v2] buildsys: Fix building with SASL on Windows

[PATCH v2 0/2] buildsys: Fix building with SASL on Windows

Philippe Mathieu-Daudé posted 2 patches 4 years, 6 months ago

Diff against v1
Download series mbox

Test docker-quick@centos7 passed

Test checkpatch passed

Test docker-clang@ubuntu passed

Test FreeBSD passed

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

Maintainers: Fam Zheng <fam@euphon.net>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>

configure                                    | 4 +++-
tests/docker/dockerfiles/debian-amd64.docker | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH v2 0/2] buildsys: Fix building with SASL on Windows

Posted by Philippe Mathieu-Daudé 4 years, 6 months ago

Fix a bug reported by Youry few months ago:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg625606.html

Since v1:
- addressed Daniel review (always define STRUCT_IOVEC_DEFINED)

The Fedora docker image already uses the libsasl since commit
8ea5962f286. Add the similar package to the Debian (host) image.

Philippe Mathieu-Daudé (2):
  configure: Fix building with SASL on Windows
  tests/docker: Install SASL library to extend code coverage on amd64

 configure                                    | 4 +++-
 tests/docker/dockerfiles/debian-amd64.docker | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

-- 
2.21.1

Re: [PATCH v2 0/2] buildsys: Fix building with SASL on Windows

Posted by Paolo Bonzini 4 years, 6 months ago

On 09/03/20 13:24, Philippe Mathieu-Daudé wrote:
> Fix a bug reported by Youry few months ago:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg625606.html
> 
> Since v1:
> - addressed Daniel review (always define STRUCT_IOVEC_DEFINED)
> 
> The Fedora docker image already uses the libsasl since commit
> 8ea5962f286. Add the similar package to the Debian (host) image.
> 
> Philippe Mathieu-Daudé (2):
>   configure: Fix building with SASL on Windows
>   tests/docker: Install SASL library to extend code coverage on amd64
> 
>  configure                                    | 4 +++-
>  tests/docker/dockerfiles/debian-amd64.docker | 1 +
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 

Queued, thanks.

Paolo