* [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency
@ 2013-09-11 16:49 Peter Maydell
2013-09-12 1:23 ` Fam Zheng
0 siblings, 1 reply; 4+ messages in thread
From: Peter Maydell @ 2013-09-11 16:49 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Paolo Bonzini, Andreas Färber, Fam Zheng,
patches
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 <peter.maydell@linaro.org>
---
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.
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency
2013-09-11 16:49 [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency Peter Maydell
@ 2013-09-12 1:23 ` Fam Zheng
2013-09-12 14:14 ` Andreas Färber
0 siblings, 1 reply; 4+ messages in thread
From: Fam Zheng @ 2013-09-12 1:23 UTC (permalink / raw)
To: Peter Maydell
Cc: qemu-trivial, Paolo Bonzini, Andreas Färber, qemu-devel,
patches
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 <peter.maydell@linaro.org>
> ---
> 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
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency
2013-09-12 1:23 ` Fam Zheng
@ 2013-09-12 14:14 ` Andreas Färber
2013-09-12 14:20 ` Peter Maydell
0 siblings, 1 reply; 4+ messages in thread
From: Andreas Färber @ 2013-09-12 14:14 UTC (permalink / raw)
To: famz, Peter Maydell; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel, patches
Am 12.09.2013 03:23, schrieb Fam Zheng:
> 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 <peter.maydell@linaro.org>
>> ---
>> 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.
If it works, fine with me:
Reviewed-by: Andreas Färber <andreas.faerber@web.de>
I don't think I have any other Cocoa patches queued - or did I miss a
more acceptable version of the command-key workaround?
Regards,
Andreas
>
> 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
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-09-12 14:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-11 16:49 [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency Peter Maydell
2013-09-12 1:23 ` Fam Zheng
2013-09-12 14:14 ` Andreas Färber
2013-09-12 14:20 ` Peter Maydell
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).