WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-ia64-devel] Re: [Xen-devel] [PATCH 2 of 2] cross-build fixes in ioe

To: Aron Griffis <aron@xxxxxx>
Subject: [Xen-ia64-devel] Re: [Xen-devel] [PATCH 2 of 2] cross-build fixes in ioemu-remote
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Thu, 31 Jul 2008 16:16:37 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 31 Jul 2008 08:16:43 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <m2n.s.1KOZmg-002Rh9@xxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Newsgroups: chiark.mail.xen.devel
References: <patchbomb.1217515898@xxxxxxxxxxxxx> <m2n.s.1KOZmg-002Rh9@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Aron Griffis writes ("[Xen-devel] [PATCH 2 of 2] cross-build fixes in 
ioemu-remote"):
> Partly fix cross-building in ioemu-remote:
> 
> - The ioemu version of cc-option doesn't take $(CC) as the first
>   argument, so all builds were getting -msse2 regardless of
>   architecture.

I'll commit this in just a moment.

> - Don't call install -s; it uses the host strip program which
>   doesn't understand cross-built binaries.  This same change was
>   in the legacy ioemu dir.

I think this should be fixed by the build environment, or possibly
$(INSTALL), being arranged to not have this bug.  If $(INSTALL) -s
doesn't work then this should be fixed.  Options include putting a
symlink to /bin/true on the path somewhere under the name `strip', or
overriding INSTALL somehow to a more clever program.

I really don't want to apply these Makefile changes.  That is the kind
of thing which makes merging painful.

Ian.

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel