docker: fix test-mingw

[Qemu-devel] [PATCH] docker: fix test-mingw

Posted by Paolo Bonzini 5 years, 2 months ago

--with-sdlabi has been removed.  Drop it from the test-mingw docker test.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 tests/docker/test-mingw | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/docker/test-mingw b/tests/docker/test-mingw
index b078f22..4b84cfe 100755
--- a/tests/docker/test-mingw
+++ b/tests/docker/test-mingw
@@ -27,8 +27,7 @@ for prefix in x86_64-w64-mingw32- i686-w64-mingw32-; do
         --enable-curl \
         --enable-vnc \
         --enable-bzip2 \
-        --enable-guest-agent \
-        --with-sdlabi=2.0
+        --enable-guest-agent
     install_qemu
     make clean
 
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH] docker: fix test-mingw

Posted by Peter Maydell 5 years, 2 months ago

On Mon, 4 Feb 2019 at 17:10, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> --with-sdlabi has been removed.  Drop it from the test-mingw docker test.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  tests/docker/test-mingw | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/tests/docker/test-mingw b/tests/docker/test-mingw
> index b078f22..4b84cfe 100755
> --- a/tests/docker/test-mingw
> +++ b/tests/docker/test-mingw
> @@ -27,8 +27,7 @@ for prefix in x86_64-w64-mingw32- i686-w64-mingw32-; do
>          --enable-curl \
>          --enable-vnc \
>          --enable-bzip2 \
> -        --enable-guest-agent \
> -        --with-sdlabi=2.0
> +        --enable-guest-agent
>      install_qemu
>      make clean

This is already in master, as commit 335dbb5de1e98c4dc7359.

thanks
-- PMM

Re: [Qemu-devel] [PATCH] docker: fix test-mingw

Posted by no-reply@patchew.org 5 years, 2 months ago

Patchew URL: https://patchew.org/QEMU/1549299863-42415-1-git-send-email-pbonzini@redhat.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Subject: [Qemu-devel] [PATCH] docker: fix test-mingw
Type: series
Message-id: 1549299863-42415-1-git-send-email-pbonzini@redhat.com

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
   335dbb5de1..773c4a6228  master     -> master
 * [new tag]               patchew/1549299863-42415-1-git-send-email-pbonzini@redhat.com -> patchew/1549299863-42415-1-git-send-email-pbonzini@redhat.com
Switched to a new branch 'test'

=== OUTPUT BEGIN ===
checkpatch.pl: no revisions returned for revlist '1'
=== OUTPUT END ===

Test command exited with code: 255


The full log is available at
http://patchew.org/logs/1549299863-42415-1-git-send-email-pbonzini@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com

Re: [Qemu-devel] [PATCH] docker: fix test-mingw

Posted by Peter Maydell 5 years, 2 months ago

On Mon, 4 Feb 2019 at 17:14, <no-reply@patchew.org> wrote:
> Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
> From https://github.com/patchew-project/qemu
>    335dbb5de1..773c4a6228  master     -> master
>  * [new tag]               patchew/1549299863-42415-1-git-send-email-pbonzini@redhat.com -> patchew/1549299863-42415-1-git-send-email-pbonzini@redhat.com
> Switched to a new branch 'test'
>
> === OUTPUT BEGIN ===
> checkpatch.pl: no revisions returned for revlist '1'
> === OUTPUT END ===

This is an interesting failure mode -- presumably it's what you
get if you post a patch which ends up without any content when
it's applied to master...

thanks
-- PMM