[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 for-2.0] vl.c: Fix OpenBSD compilation issue
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH V2 for-2.0] vl.c: Fix OpenBSD compilation issue due to namespace collisions |
Date: |
Wed, 19 Mar 2014 14:38:36 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
Am 18.03.2014 16:26, schrieb Marcel Apfelbaum:
> Machine rewriting added MACHINE macro which is
> already in use by other OpenBSD library.
> Since qemu/sockets.h exposes the OpenBSD namespace,
> the minimalistic approach is to add it as the first QEMU include.
>
> Reported-by: Brad Smith <address@hidden>
> Signed-off-by: Marcel Apfelbaum <address@hidden>
> ---
> V1 -> V2
> #undef MACHINE to avoid introducing warnings for redefining the macro.
>
> include/hw/boards.h | 1 +
> vl.c | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/hw/boards.h b/include/hw/boards.h
> index 7bd2ea7..dd2c70d 100644
> --- a/include/hw/boards.h
> +++ b/include/hw/boards.h
> @@ -54,6 +54,7 @@ struct QEMUMachine {
> int qemu_register_machine(QEMUMachine *m);
>
> #define TYPE_MACHINE "machine"
> +#undef MACHINE /* BSD defines it and QEMU does not use it */
> #define MACHINE(obj) \
> OBJECT_CHECK(MachineState, (obj), TYPE_MACHINE)
> #define MACHINE_GET_CLASS(obj) \
> diff --git a/vl.c b/vl.c
> index f0fe48b..9f86e68 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -58,6 +58,7 @@ int main(int argc, char **argv)
>
> #include <glib.h>
>
> +#include "qemu/sockets.h"
> #include "hw/hw.h"
> #include "hw/boards.h"
> #include "hw/usb.h"
Thanks, I've queued this for testing but would probably like to add a
comment explaining why this is placed there.
https://github.com/afaerber/qemu-cpu/commits/qom-next
Andreas
> @@ -103,7 +104,6 @@ int main(int argc, char **argv)
>
> #include "disas/disas.h"
>
> -#include "qemu/sockets.h"
>
> #include "slirp/libslirp.h"
>
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg