[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/12] slirp updates
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 00/12] slirp updates |
Date: |
Thu, 7 Mar 2019 11:27:38 +0000 |
On Wed, 6 Mar 2019 at 23:59, Samuel Thibault
<address@hidden> wrote:
>
> The following changes since commit 32694e98b8d7a246345448a8f707d2e11d6c65e2:
>
> Merge remote-tracking branch
> 'remotes/ehabkost/tags/machine-next-pull-request' into staging (2019-03-06
> 18:52:19 +0000)
>
> are available in the Git repository at:
>
> https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault
>
> for you to fetch changes up to dd9eff6c839db3996c157f0a6a4e18f95683e58c:
>
> slirp: remove QEMU Makefile.objs (2019-03-07 00:12:34 +0100)
>
> ----------------------------------------------------------------
Hi; this fails to build on FreeBSD, I'm afraid:
In file included from /var/tmp/qemu-test.Sn5gQz/slirp/src/stream.c:24:
In file included from /var/tmp/qemu-test.Sn5gQz/slirp/src/stream.h:4:
/var/tmp/qemu-test.Sn5gQz/slirp/src/libslirp.h:29:19: warning: type
specifier missing, defaults to 'int' [-Wimplicit-int]
typedef ssize_t (*SlirpReadCb)(void *buf, size_t len, void *opaque);
^
/var/tmp/qemu-test.Sn5gQz/slirp/src/libslirp.h:29:9: warning: type
specifier missing, defaults to 'int' [-Wimplicit-int]
typedef ssize_t (*SlirpReadCb)(void *buf, size_t len, void *opaque);
~~~~~~~ ^
/var/tmp/qemu-test.Sn5gQz/slirp/src/libslirp.h:29:17: error: function
cannot return function type 'int (void *, size_t, void *)' (aka 'int
(void *, unsigned
long, void *)')
typedef ssize_t (*SlirpReadCb)(void *buf, size_t len, void *opaque);
^
/var/tmp/qemu-test.Sn5gQz/slirp/src/libslirp.h:30:32: error: function
cannot return function type 'ssize_t' (aka 'int (int *)')
typedef ssize_t (*SlirpWriteCb)(const void *buf, size_t len, void *opaque);
^
/var/tmp/qemu-test.Sn5gQz/slirp/src/libslirp.h:108:22: error: unknown
type name 'SlirpReadCb'
SlirpReadCb read_cb, void *opaque);
^
In file included from /var/tmp/qemu-test.Sn5gQz/slirp/src/stream.c:24:
/var/tmp/qemu-test.Sn5gQz/slirp/src/stream.h:7:5: error: unknown type
name 'SlirpReadCb'
SlirpReadCb read_cb;
^
2 warnings and 4 errors generated.
I think this is a missing #include of <sys/types.h> which is where
ssize_t comes from -- on the other OSes it likely gets dragged in
by default.
thanks
-- PMM
- [Qemu-devel] [PULL 08/12] slirp: use "slirp_" prefix for inet_aton() win32 implementation, (continued)
- [Qemu-devel] [PULL 08/12] slirp: use "slirp_" prefix for inet_aton() win32 implementation, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 10/12] slirp: add a standalone Makefile, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 02/12] slirp: check for ioctlsocket error and 0-length udp payload., Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 03/12] slirp: check sscanf result when emulating ident, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 06/12] slirp: adapt a subset of QEMU vmstate code, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 12/12] slirp: remove QEMU Makefile.objs, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 09/12] slirp: move sources to src/ subdirectory, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 11/12] build-sys: link with slirp as an external project, Samuel Thibault, 2019/03/06
- [Qemu-devel] [PULL 07/12] slirp: use libslirp migration code, Samuel Thibault, 2019/03/06
- Re: [Qemu-devel] [PULL 00/12] slirp updates, no-reply, 2019/03/06
- Re: [Qemu-devel] [PULL 00/12] slirp updates,
Peter Maydell <=