qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/
@ 2012-06-09 11:56 Andreas Färber
  2012-06-11  5:51 ` Paolo Bonzini
  2012-06-12  9:50 ` [Qemu-devel] [Qemu-trivial] " Stefan Hajnoczi
  0 siblings, 2 replies; 3+ messages in thread
From: Andreas Färber @ 2012-06-09 11:56 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, pbonzini, Andreas Färber

In *-*-user/ build dirs CONFIG_USER_ONLY is defined via config-target.h.
In libuser/ it is not defined.

Add it via QEMU_CFLAGS.

Signed-off-by: Andreas Färber <afaerber@suse.de>
---
 Soon needed for cpu.c.

 Makefile.user |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/Makefile.user b/Makefile.user
index b717820..ab70eb3 100644
--- a/Makefile.user
+++ b/Makefile.user
@@ -10,6 +10,7 @@ $(call set-vpath, $(SRC_PATH))
 
 QEMU_CFLAGS+=-I..
 QEMU_CFLAGS += -I$(SRC_PATH)/include
+QEMU_CFLAGS += -DCONFIG_USER_ONLY
 
 include $(SRC_PATH)/Makefile.objs
 
-- 
1.7.7

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/
  2012-06-09 11:56 [Qemu-devel] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/ Andreas Färber
@ 2012-06-11  5:51 ` Paolo Bonzini
  2012-06-12  9:50 ` [Qemu-devel] [Qemu-trivial] " Stefan Hajnoczi
  1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2012-06-11  5:51 UTC (permalink / raw)
  To: Andreas Färber; +Cc: qemu-trivial, qemu-devel



----- Messaggio originale -----
> Da: "Andreas Färber" <afaerber@suse.de>
> A: qemu-devel@nongnu.org
> Cc: qemu-trivial@nongnu.org, pbonzini@redhat.com, "Andreas Färber" <afaerber@suse.de>
> Inviato: Sabato, 9 giugno 2012 13:56:42
> Oggetto: [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/
>
> In *-*-user/ build dirs CONFIG_USER_ONLY is defined via
> config-target.h.
> In libuser/ it is not defined.
>
> Add it via QEMU_CFLAGS.
>
> Signed-off-by: Andreas Färber <afaerber@suse.de>

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>

Paolo

> ---
>  Soon needed for cpu.c.
>
>  Makefile.user |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/Makefile.user b/Makefile.user
> index b717820..ab70eb3 100644
> --- a/Makefile.user
> +++ b/Makefile.user
> @@ -10,6 +10,7 @@ $(call set-vpath, $(SRC_PATH))
>
>  QEMU_CFLAGS+=-I..
>  QEMU_CFLAGS += -I$(SRC_PATH)/include
> +QEMU_CFLAGS += -DCONFIG_USER_ONLY
>
>  include $(SRC_PATH)/Makefile.objs
>
> --
> 1.7.7
>
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [Qemu-trivial] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/
  2012-06-09 11:56 [Qemu-devel] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/ Andreas Färber
  2012-06-11  5:51 ` Paolo Bonzini
@ 2012-06-12  9:50 ` Stefan Hajnoczi
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2012-06-12  9:50 UTC (permalink / raw)
  To: Andreas Färber; +Cc: qemu-trivial, pbonzini, qemu-devel

On Sat, Jun 09, 2012 at 01:56:42PM +0200, Andreas Färber wrote:
> In *-*-user/ build dirs CONFIG_USER_ONLY is defined via config-target.h.
> In libuser/ it is not defined.
> 
> Add it via QEMU_CFLAGS.
> 
> Signed-off-by: Andreas Färber <afaerber@suse.de>
> ---
>  Soon needed for cpu.c.
> 
>  Makefile.user |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-06-12  9:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-09 11:56 [Qemu-devel] [PATCH] Makefile.user: Define CONFIG_USER_ONLY for libuser/ Andreas Färber
2012-06-11  5:51 ` Paolo Bonzini
2012-06-12  9:50 ` [Qemu-devel] [Qemu-trivial] " Stefan Hajnoczi

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).