[kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-29 Thread Hollis Blanchard
Move libkvm into its own directory. No functional changes. Signed-off-by: Hollis Blanchard <[EMAIL PROTECTED]> --- I've build-tested this pretty thoroughly on x86(32). Changes from v1: update libkvm.h #include guard macro. diff --git a/Makefile b/Makefile --- a/Makefile +++ b/Makefile @@ -5,16

Re: [kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-30 Thread Avi Kivity
Hollis Blanchard wrote: > Move libkvm into its own directory. No functional changes. > > Applied, thanks. -- Any sufficiently difficult bug is indistinguishable from a feature. - This SF.net email is sponsored by: Splun

Re: [kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-31 Thread Cam Macdonell
Hi Hollis and Avi, I run into a compile problem with libkvm when compiling kvm-49. It caused the compile problem mentioned in my email yesterday. It has to with the line in libkvm/Makefile CFLAGS += -I $(KERNELDIR)/include when compiling kvm-49 (git or snapshot), this include points to my

Re: [kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-31 Thread Hollis Blanchard
I think the KERNELDIR change is a red herring: you probably ran configure with different arguments. I notice that the kvm-49 build line below is missing -m64. Try this: diff --git a/libkvm/config-x86_64.mak b/libkvm/config-x86_64.mak --- a/libkvm/config-x86_64.mak +++ b/libkvm/config-x86_64.mak @

Re: [kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-31 Thread Cam Macdonell
Hollis Blanchard wrote: > I think the KERNELDIR change is a red herring: you probably ran > configure with different arguments. Hi Hollis, The only one I run with is --prefix. It seems Avi latest patches fix the problem (although I'm not sure how): With Avi's latest changes, this now the compi

Re: [kvm-devel] [PATCH] [RESEND] Move libkvm (v2)

2007-10-31 Thread Avi Kivity
Cam Macdonell wrote: > > Hi Hollis and Avi, > > I run into a compile problem with libkvm when compiling kvm-49. It > caused the compile problem mentioned in my email yesterday. > > It has to with the line in libkvm/Makefile > > CFLAGS += -I $(KERNELDIR)/include > > when compiling kvm-49 (git or