From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Egger Subject: Re: [PATCH] fix build error with seabios Date: Wed, 25 Apr 2012 10:43:45 +0200 Message-ID: <4F97B941.7000108@amd.com> References: <4F8836C9.9080502@amd.com> <20374.57450.417199.89376@mariner.uk.xensource.com> <1335341262.4881.15.camel@dagon.hellion.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1335341262.4881.15.camel@dagon.hellion.org.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Ian Jackson , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On 04/25/12 10:07, Ian Campbell wrote: > On Tue, 2012-04-24 at 18:18 +0100, Ian Jackson wrote: >> Christoph Egger writes ("[Xen-devel] [PATCH] fix build error with seabios"): >>> >>> Pass PYTHON down to seabios, so seabios will >>> use same python binary as whole xen tree does. >>> Fixes build error on NetBSD. >> >> Ian, does this look sensible to you ? > > It exports $(PYTHON) to all subdirs of tools/firmware, but I guess that > is OK, so we might as well take this now. Thanks. > Does > subdirs-seabios: PYTHON=$(PYTHON) > (or something similar) work? Might be a better option in the future No, this doesn't work. > >> >>> Signed-off-by: Christoph Egger > > Acked-by: Ian Campbell > >>> >>> ---------------------------------------------------------------------- >>> diff -r ab552da976a3 tools/firmware/Makefile >>> --- a/tools/firmware/Makefile Wed Apr 11 18:28:33 2012 +0200 >>> +++ b/tools/firmware/Makefile Fri Apr 13 16:22:23 2012 +0200 >>> @@ -32,7 +32,7 @@ ifeq ($(CONFIG_ROMBIOS),y) >>> false ; \ >>> fi >>> endif >>> - $(MAKE) subdirs-$@ >>> + $(MAKE) PYTHON=$(PYTHON) subdirs-$@ >>> >>> >>> .PHONY: install >>> >>> ---------------------------------------------------------------------- >>> _______________________________________________ >>> Xen-devel mailing list >>> Xen-devel@lists.xen.org >>> http://lists.xen.org/xen-devel > > > -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632