qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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/9] Introduce I/O channels framework
Date: Thu, 17 Dec 2015 15:30:22 +0000	[thread overview]
Message-ID: <20151217153022.GI16238@redhat.com> (raw)
In-Reply-To: <CAFEAcA9g5qtn0Cm=Ttb8gRUrAiNT0A6dHR7wuFV7Q0NYST-uxA@mail.gmail.com>

On Thu, Dec 17, 2015 at 03:04:33PM +0000, Peter Maydell wrote:
> On 17 December 2015 at 13:45, Daniel P. Berrange <berrange@redhat.com> wrote:
> > The following changes since commit c1a5f950cdeeaea6a835b2b33f040a0e62c18662:
> >
> >   Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-12-17' into staging (2015-12-17 11:50:46 +0000)
> >
> > are available in the git repository at:
> >
> >   git://github.com/berrange/qemu tags/pull-io-channel-base-2015-12-17-1
> >
> > for you to fetch changes up to 37c41c79730f7535c54f304e5c6e89fbf2cfe0e5:
> >
> >   io: add QIOChannelBuffer class (2015-12-17 13:26:04 +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
> 
> Hi. I'm afraid this fails to build on OSX:
> 
> In file included from /Users/pm215/src/qemu-for-merges/io/channel.c:21:
> /Users/pm215/src/qemu-for-merges/include/io/channel.h:127:5: error:
> type name requires a specifier or qualifier
>     off64_t (*io_seek)(QIOChannel *ioc,

[snip]

> 1 warning and 9 errors generated.

Ok, will repost with  s/off_t/off64_t/ since QEMU builds with
-D_FILE_OFFSET_BITS=64 meaning both off_t and off64_t are the
same size anyway.


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

      reply	other threads:[~2015-12-17 15:30 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-17 13:45 [Qemu-devel] [PULL v1 0/9] Introduce I/O channels framework Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 1/9] io: add abstract QIOChannel classes Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 2/9] io: add helper module for creating watches on FDs Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 3/9] io: add QIOTask class for async operations Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 4/9] io: add QIOChannelSocket class Daniel P. Berrange
2015-12-22 14:44   ` Peter Maydell
2015-12-22 14:51     ` Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 5/9] io: add QIOChannelFile class Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 6/9] io: add QIOChannelTLS class Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 7/9] io: add QIOChannelWebsock class Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 8/9] io: add QIOChannelCommand class Daniel P. Berrange
2015-12-17 13:45 ` [Qemu-devel] [PULL v1 9/9] io: add QIOChannelBuffer class Daniel P. Berrange
2015-12-17 15:04 ` [Qemu-devel] [PULL v1 0/9] Introduce I/O channels framework Peter Maydell
2015-12-17 15:30   ` 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=20151217153022.GI16238@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).