From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 03/13] Config.mk: add new macro buildmakevars2header Date: Tue, 26 Aug 2014 21:02:27 +0100 Message-ID: <1409083347.28009.22.camel@citrix.com> References: <1406538316-29087-1-git-send-email-olaf@aepfle.de> <1406538316-29087-4-git-send-email-olaf@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1406538316-29087-4-git-send-email-olaf@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Olaf Hering Cc: Keir Fraser , Stefano Stabellini , Tim Deegan , Ian Jackson , xen-devel@lists.xen.org, Jan Beulich , Samuel Thibault List-Id: xen-devel@lists.xenproject.org On Mon, 2014-07-28 at 11:05 +0200, Olaf Hering wrote: > This macro is similar to buildmakevars2file, it just creates a C header > file instead of shell style syntax. Upcoming changes will use this macro > in libxl and libxc. > > Note: if target ($(1)) is marked as .PHONY everything that depends on > target will be constantly rebuild. The reason for that seems to be that > make will internally drop target. As a result target is non-existant, "existent" > which triggers a rebuild. With the intermediate target.tmp this will not > happen because target changes only if target.tmp really changes. Similar to the previous patch do you really need $(1).tmp as a target as well as a (non-.PHONY) $(1) target? Ian.