[Qemu-devel] [PATCH] configure: let it find xen headers and libs

[Qemu-devel] [PATCH] configure: let it find xen headers and libs

[Christoph Egger]

[-- Attachment #1: Type: text/plain, Size: 449 bytes --]


Hi!

Attached patch lets configure find xen headers and libs
with --extra-cflags and --extra-ldlfags option.

Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>

-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

[-- Attachment #2: qemu_xen_configure.diff --]
[-- Type: text/x-diff, Size: 496 bytes --]

diff --git a/configure b/configure
index 8160bed..5ae2aba 100755
--- a/configure
+++ b/configure
@@ -880,7 +880,7 @@ if test "$xen" = "yes" ; then
 #include <xs.h>
 int main(void) { xs_daemon_open(); xc_interface_open(); return 0; }
 EOF
-  if $cc $CFLAGS $ARCH_CFLAGS -c -o $TMPO $TMPC $LDFLAGS $xen_libs 2> /dev/null > /dev/null ; then
+  if $cc $CFLAGS $ARCH_CFLAGS $EXTRA_CFLAGS -c -o $TMPO $TMPC $LDFLAGS $EXTRA_LDFLAGS $xen_libs 2> /dev/null > /dev/null ; then
     :
   else
     xen="no"

[Qemu-devel] Re: [PATCH] configure: let it find xen headers and libs

[Juan Quintela]

Christoph Egger <Christoph.Egger@amd.com> wrote:
> Hi!
>
> Attached patch lets configure find xen headers and libs
> with --extra-cflags and --extra-ldlfags option.
>
> Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>

One of the patches in my last series should have fixed this one, please
take a look :)  Fixed for all tests, not only xen.

Later, Juan.

[Qemu-devel] Re: [PATCH] configure: let it find xen headers and libs

[Christoph Egger]

On Thursday 30 July 2009 16:42:03 Juan Quintela wrote:
> Christoph Egger <Christoph.Egger@amd.com> wrote:
> > Hi!
> >
> > Attached patch lets configure find xen headers and libs
> > with --extra-cflags and --extra-ldlfags option.
> >
> > Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
>
> One of the patches in my last series should have fixed this one, please
> take a look :)  Fixed for all tests, not only xen.

I vote for it to go upstream then. Avoids duplicate work. :)

Christoph

-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632