qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: "Daniel P. Berrange" <berrange@redhat.com>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL v2 0/9] Introduce I/O channels framework
Date: Thu, 17 Dec 2015 16:40:47 +0000	[thread overview]
Message-ID: <CAFEAcA9FfVcXtouzuLmdmYqPED8jp1JvH+PC2A9csZdKEaeVvw@mail.gmail.com> (raw)
In-Reply-To: <1450366809-9800-1-git-send-email-berrange@redhat.com>

On 17 December 2015 at 15:40, Daniel P. Berrange <berrange@redhat.com> wrote:
> The following changes since commit e5fbe28e5424d26fc2c25d0a7ecb927d3c80d5e8:
>
>   Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20151217-1' into staging (2015-12-17 13:38:34 +0000)
>
> are available in the git repository at:
>
>   git://github.com/berrange/qemu tags/pull-io-channel-base-2015-12-17-2
>
> for you to fetch changes up to 65499bafaf39b75c9787ef39fffeb3ff9fe48424:
>
>   io: add QIOChannelBuffer class (2015-12-17 15:28:33 +0000)
>
> ----------------------------------------------------------------
> Merge I/O channels base classes
>
> ----------------------------------------------------------------
> Daniel P. Berrange (9):
>       io: add abstract QIOChannel classes
>       io: add helper module for creating watches on FDs
>       io: add QIOTask class for async operations
>       io: add QIOChannelSocket class
>       io: add QIOChannelFile class
>       io: add QIOChannelTLS class
>       io: add QIOChannelWebsock class
>       io: add QIOChannelCommand class
>       io: add QIOChannelBuffer class

Still fails on OSX I'm afraid:

/Users/pm215/src/qemu-for-merges/io/channel-buffer.c:211:5: warning:
implicit declaration of function 'g_source_set_name' is invalid
 in C99 [-Wimplicit-function-declaration]
    g_source_set_name(source, "QIOChannelBuffer");
    ^

/Users/pm215/src/qemu-for-merges/io/channel-file.c:164:11: warning:
implicit declaration of function 'lseek64' is invalid in C99 [-W
implicit-function-declaration]
    ret = lseek64(fioc->fd, offset, whence);
          ^

/Users/pm215/src/qemu-for-merges/io/channel-watch.c:155:5: warning:
implicit declaration of function 'g_source_set_name' is invalid in C99
[-Wimplicit-function-declaration]
    g_source_set_name(source, "QIOChannelFD");
    ^

/Users/pm215/src/qemu-for-merges/io/channel-websock.c:923:5: warning:
implicit declaration of function 'g_source_set_name' is invalid in C99
[-Wimplicit-function-declaration]
    g_source_set_name(source, "QIOChannelWebsock");
    ^
  CC    iothread.o

(and then the link fails because those functions aren't present).

Glib's g_source_set_name was only introduced in 2.26,
and we have to support 2.22.

lseek64 doesn't exist on OSX.

thanks
-- PMM

  parent reply	other threads:[~2015-12-17 16:41 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-17 15:40 [Qemu-devel] [PULL v2 0/9] Introduce I/O channels framework Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 1/9] io: add abstract QIOChannel classes Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 2/9] io: add helper module for creating watches on FDs Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 3/9] io: add QIOTask class for async operations Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 4/9] io: add QIOChannelSocket class Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 5/9] io: add QIOChannelFile class Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 6/9] io: add QIOChannelTLS class Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 7/9] io: add QIOChannelWebsock class Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 8/9] io: add QIOChannelCommand class Daniel P. Berrange
2015-12-17 15:40 ` [Qemu-devel] [PULL v2 9/9] io: add QIOChannelBuffer class Daniel P. Berrange
2015-12-17 16:40 ` Peter Maydell [this message]
2015-12-17 16:44   ` [Qemu-devel] [PULL v2 0/9] Introduce I/O channels framework Paolo Bonzini
2015-12-17 16:47   ` Daniel P. Berrange
2015-12-17 18:32     ` Paolo Bonzini

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=CAFEAcA9FfVcXtouzuLmdmYqPED8jp1JvH+PC2A9csZdKEaeVvw@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=berrange@redhat.com \
    --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).