[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17 |
Date: |
Tue, 18 Jun 2019 08:07:42 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On Mon, 17 Jun 2019 at 19:51, Markus Armbruster <address@hidden> wrote:
>>
>> The following changes since commit 076243ffe6c1b687e9e6d98348c3bf3398df78f3:
>>
>> Merge remote-tracking branch 'remotes/pmaydell/tags/pull-docs-20190617'
>> into staging (2019-06-17 16:41:25 +0100)
>>
>> are available in the Git repository at:
>>
>> git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2019-06-17
>>
>> for you to fetch changes up to 092b8737c5e7695c4b9caa3b4eedc66763632755:
>>
>> vl: Deprecate -mon pretty=... for HMP monitors (2019-06-17 20:36:56 +0200)
>>
>> ----------------------------------------------------------------
>> Monitor patches for 2019-06-17
>>
>
> Hi; I'm afraid this doesn't compile:
>
> Most hosts (ppc64, s390, aarch64, aarch32, osx, and possibly
> the others too though they haven't failed immediately) fail
> something like this:
>
> /home/pm215/qemu/monitor/misc.c: In function ‘netdev_del_completion’:
> /home/pm215/qemu/monitor/misc.c:2165:9: error: implicit declaration of
> function ‘qemu_find_opts_err’ [-Werror=implicit-function-declaration]
> opts = qemu_opts_find(qemu_find_opts_err("netdev", NULL), name);
> ^
> /home/pm215/qemu/monitor/misc.c:2165:9: error: nested extern
> declaration of ‘qemu_find_opts_err’ [-Werror=nested-externs]
> /home/pm215/qemu/monitor/misc.c:2165:9: error: passing argument 1 of
> ‘qemu_opts_find’ makes pointer from integer without a cast [-Werror]
> In file included from /home/pm215/qemu/monitor/misc.c:64:0:
> /home/pm215/qemu/include/qemu/option.h:105:11: note: expected ‘struct
> QemuOptsList *’ but argument is of type ‘int’
> QemuOpts *qemu_opts_find(QemuOptsList *list, const char *id);
> ^
> cc1: all warnings being treated as errors
Yup, my fault: I fell into the trap laid by include/ui/qemu-spice.h,
which includes qemu/config-file.h only when CONFIG_SPICE.
> windows is a bit different:
>
> /home/petmay01/qemu-for-merges/monitor/hmp.c: In function 'file_completion':
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1113:5: error: unknown
> type name 'DIR'
> DIR *ffs;
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1135:11: error: implicit
> declaration of function 'opendir'
> [-Werror=implicit-function-declaration]
> ffs = opendir(path);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1135:5: error: nested
> extern declaration of 'opendir' [-Werror=nested-externs]
> ffs = opendir(path);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1135:9: error: assignment
> makes pointer from integer without a cast [-Werror=int-conversion]
> ffs = opendir(path);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1141:13: error: implicit
> declaration of function 'readdir'
> [-Werror=implicit-function-declaration]
> d = readdir(ffs);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1141:9: error: nested
> extern declaration of 'readdir' [-Werror=nested-externs]
> d = readdir(ffs);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1141:11: error:
> assignment makes pointer from integer without a cast
> [-Werror=int-conversion]
> d = readdir(ffs);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1146:21: error:
> dereferencing pointer to incomplete type 'struct dirent'
> if (strcmp(d->d_name, ".") == 0 || strcmp(d->d_name, "..") == 0) {
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1166:5: error: implicit
> declaration of function 'closedir'
> [-Werror=implicit-function-declaration]
> closedir(ffs);
> ^
> /home/petmay01/qemu-for-merges/monitor/hmp.c:1166:5: error: nested
> extern declaration of 'closedir' [-Werror=nested-externs]
Here, Kevin fell into the trap laid by GLib, which includes <dirent.h>
only #ifdef G_OS_UNIX.
v2 coming up.
- [Qemu-devel] [PULL 02/16] monitor: Remove unused password prompting fields, (continued)
- [Qemu-devel] [PULL 02/16] monitor: Remove unused password prompting fields, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 11/16] monitor: Split out monitor/qmp.c, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 15/16] monitor: Replace monitor_init() with monitor_init_{hmp, qmp}(), Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 05/16] monitor: Create MonitorHMP with readline state, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 16/16] vl: Deprecate -mon pretty=... for HMP monitors, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 07/16] monitor: Rename HMP command type and tables, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 13/16] monitor: Split out monitor/monitor.c, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 12/16] monitor: Split out monitor/hmp.c, Markus Armbruster, 2019/06/17
- [Qemu-devel] [PULL 10/16] monitor: Create monitor-internal.h with common definitions, Markus Armbruster, 2019/06/17
- Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17, Peter Maydell, 2019/06/17
- Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17,
Markus Armbruster <=
- Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17, no-reply, 2019/06/17
- Re: [Qemu-devel] [PULL 00/16] Monitor patches for 2019-06-17, no-reply, 2019/06/17