From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57562) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK7fM-0005wg-Fk for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:14:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VK7fF-0000Ch-Md for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:14:40 -0400 Received: from mout.web.de ([212.227.15.3]:56544) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VK7fF-0000CS-Cy for qemu-devel@nongnu.org; Thu, 12 Sep 2013 10:14:33 -0400 Received: from envy.site ([195.135.221.2]) by smtp.web.de (mrweb102) with ESMTPSA (Nemesis) id 0LqDUa-1Vonif3niy-00dk4V for ; Thu, 12 Sep 2013 16:14:31 +0200 Message-ID: <5231CC3F.30001@web.de> Date: Thu, 12 Sep 2013 16:14:23 +0200 From: =?ISO-8859-1?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1378918197-9036-1-git-send-email-peter.maydell@linaro.org> <20130912012318.GA9134@T430s.nay.redhat.com> In-Reply-To: <20130912012318.GA9134@T430s.nay.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH] ui/Makefile.objs: delete unnecessary cocoa.o dependency List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: famz@redhat.com, Peter Maydell Cc: qemu-trivial@nongnu.org, Paolo Bonzini , qemu-devel@nongnu.org, patches@linaro.org 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 >> --- >> 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 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 >>