[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o
From: |
Fam Zheng |
Subject: |
Re: [Qemu-trivial] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency |
Date: |
Thu, 12 Sep 2013 09:23:18 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, 09/11 17:49, Peter Maydell wrote:
> Delete an unnecessary dependency for cocoa.o; we already have
> a general rule that tells Make that we can build a .o file
> from a .m source using an ObjC compiler, so this specific
> rule is unnecessary. Further, it is using the dubious construct
> "$(SRC_PATH)/$(obj)" to get at the source directory, which will
> break when $(obj) is redefined as part of the preparation for
> per-object library support.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> This needs to go in before Fam's patchset for modules
> can be applied, either via a cocoa maintainer queue or via
> -trivial or just as a first patch on the front of Fam's
> patches if they would otherwise be being held up by this.
>
Sure, I'll apply to my tree and rebase if it's merged.
Thanks,
Fam
> ui/Makefile.objs | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/ui/Makefile.objs b/ui/Makefile.objs
> index 6ddc0de..f33be47 100644
> --- a/ui/Makefile.objs
> +++ b/ui/Makefile.objs
> @@ -17,6 +17,4 @@ common-obj-$(CONFIG_GTK) += gtk.o x_keymap.o
>
> $(obj)/sdl.o $(obj)/sdl_zoom.o: QEMU_CFLAGS += $(SDL_CFLAGS)
>
> -$(obj)/cocoa.o: $(SRC_PATH)/$(obj)/cocoa.m
> -
> $(obj)/gtk.o: QEMU_CFLAGS += $(GTK_CFLAGS) $(VTE_CFLAGS)
> --
> 1.7.11.4
>