From: "Daniel P. Berrange" <berrange@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL v1 0/3] Merge qio 2017/02/27
Date: Mon, 27 Feb 2017 15:34:05 +0000 [thread overview]
Message-ID: <20170227153405.GP18219@redhat.com> (raw)
In-Reply-To: <CAFEAcA-BMy5nu1yUTRu0uZYuZSzH_Ri8YFkhNRbmFo9KEtAZRg@mail.gmail.com>
On Mon, Feb 27, 2017 at 03:33:02PM +0000, Peter Maydell wrote:
> On 27 February 2017 at 13:35, Daniel P. Berrange <berrange@redhat.com> wrote:
> > The following changes since commit d992f2f1368ceb92e6bfd8efece174110f4236ff:
> >
> > Merge remote-tracking branch 'remotes/artyom/tags/pull-sun4v-20170226' into staging (2017-02-26 22:40:23 +0000)
> >
> > are available in the git repository at:
> >
> > git://github.com/berrange/qemu tags/pull-qio-2017-02-27-1
> >
> > for you to fetch changes up to acd22400d1dce27bffc94be7663e1ff1c716abe8:
> >
> > tests: fix leaks in test-io-channel-command (2017-02-27 12:42:21 +0000)
> >
> > ----------------------------------------------------------------
> > Merge qio 2017/02/27 v1
> >
> > ----------------------------------------------------------------
>
> Fails to build, w32:
>
> /home/petmay01/linaro/qemu-for-merges/io/channel-websock.c: In
> function ‘qio_channel_websock_handshake_entry’:
> /home/petmay01/linaro/qemu-for-merges/io/channel-websock.c:118:13:
> error: implicit declaration of function ‘strcasestr’
> [-Werror=implicit-function-declaration]
> begin = strcasestr(handshake, line);
> ^
> /home/petmay01/linaro/qemu-for-merges/io/channel-websock.c:118:5:
> error: nested extern declaration of ‘strcasestr’
> [-Werror=nested-externs]
> begin = strcasestr(handshake, line);
> ^
> /home/petmay01/linaro/qemu-for-merges/io/channel-websock.c:118:11:
> error: assignment makes pointer from integer without a cast
> [-Werror=int-conversion]
> begin = strcasestr(handshake, line);
> ^
> cc1: all warnings being treated as errors
>
> mingw doesn't have strcasestr(), I think.
Yep, man page marks it as a gnu extension too
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|
prev parent reply other threads:[~2017-02-27 15:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-27 13:35 [Qemu-devel] [PULL v1 0/3] Merge qio 2017/02/27 Daniel P. Berrange
2017-02-27 13:35 ` [Qemu-devel] [PULL v1 1/3] io: fix decoding when multiple websockets frames arrive at once Daniel P. Berrange
2017-02-27 13:35 ` [Qemu-devel] [PULL v1 2/3] io: ignore case in WebSocket HTTP header Daniel P. Berrange
2017-02-27 13:35 ` [Qemu-devel] [PULL v1 3/3] tests: fix leaks in test-io-channel-command Daniel P. Berrange
2017-02-27 14:02 ` [Qemu-devel] [PULL v1 0/3] Merge qio 2017/02/27 no-reply
2017-02-27 15:33 ` Peter Maydell
2017-02-27 15:34 ` Daniel P. Berrange [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170227153405.GP18219@redhat.com \
--to=berrange@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).