Bug#402249: [Pkg-xen-devel] Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-18 Thread Guido Guenther
Hi Basti,
On Mon, Dec 17, 2007 at 05:56:01PM +0100, Bastian Blank wrote:
 On Sat, Dec 15, 2007 at 07:40:07PM +0100, Guido Guenther wrote:
  Dear Debian Xen maintainers,
  for your convenience I attach Richard's patch directly. It simply
  installs the necessary headers so we're able to enable xen support in
  libvirt for the default builds. Please apply.
 
 xen-utils is not installable on autobuilders. Don't even try to
 build-depend against.
Since libvirt would depend on xen-utils it would make libvirt
uninstallable on autobuilders too so only including the headers doesn't
look like a solution. Why don't you want xen-utils on the autobuilders?
Will the Debian Xen team split out libxenstore as Ubuntu does?
Cheers,
 -- Guido



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#402249: [Pkg-xen-devel] Bug#402249: Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-18 Thread Marco Sinhoreli
Hello Bastian, Guido, Richard,

I think that, split out libxenstore is in debian packaging model, and
is the best alternate to provide the headers into xen package.
According to my knoledge, maybe I'm wrong, isolate the headers as
Ubuntu does, is in debian police. This way isn't simple, because we'll
need to do a big changes in Xen package. Well, I'll look the xen
package structure to know what need to do.

Regards,


On Dec 18, 2007 6:45 AM, Guido Guenther [EMAIL PROTECTED] wrote:
 Hi Basti,
 On Mon, Dec 17, 2007 at 05:56:01PM +0100, Bastian Blank wrote:
  On Sat, Dec 15, 2007 at 07:40:07PM +0100, Guido Guenther wrote:
   Dear Debian Xen maintainers,
   for your convenience I attach Richard's patch directly. It simply
   installs the necessary headers so we're able to enable xen support in
   libvirt for the default builds. Please apply.
 
  xen-utils is not installable on autobuilders. Don't even try to
  build-depend against.
 Since libvirt would depend on xen-utils it would make libvirt
 uninstallable on autobuilders too so only including the headers doesn't
 look like a solution. Why don't you want xen-utils on the autobuilders?
 Will the Debian Xen team split out libxenstore as Ubuntu does?
 Cheers,
  -- Guido




 ___
 Pkg-xen-devel mailing list
 [EMAIL PROTECTED]
 http://lists.alioth.debian.org/mailman/listinfo/pkg-xen-devel




-- 
Marco Sinhoreli



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#402249: [Pkg-xen-devel] Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-17 Thread Bastian Blank
On Sat, Dec 15, 2007 at 07:40:07PM +0100, Guido Guenther wrote:
 Dear Debian Xen maintainers,
 for your convenience I attach Richard's patch directly. It simply
 installs the necessary headers so we're able to enable xen support in
 libvirt for the default builds. Please apply.

xen-utils is not installable on autobuilders. Don't even try to
build-depend against.

Bastian

-- 
Murder is contrary to the laws of man and God.
-- M-5 Computer, The Ultimate Computer, stardate 4731.3



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#402249: [Pkg-xen-devel] Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-17 Thread Bastian Blank
On Sat, Dec 15, 2007 at 07:40:07PM +0100, Guido Guenther wrote:
 for your convenience I attach Richard's patch directly. It simply
 installs the necessary headers so we're able to enable xen support in
 libvirt for the default builds. Please apply.

libvirt redefines several parts of this headers. So they can include the
missing pieces also.

Bastian

-- 
You canna change the laws of physics, Captain; I've got to have thirty minutes!



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#402249: [Pkg-xen-devel] Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-17 Thread Richard W.M. Jones

Bastian Blank wrote:

On Sat, Dec 15, 2007 at 07:40:07PM +0100, Guido Guenther wrote:

for your convenience I attach Richard's patch directly. It simply
installs the necessary headers so we're able to enable xen support in
libvirt for the default builds. Please apply.


libvirt redefines several parts of this headers. So they can include the
missing pieces also.


We include redefinitions in a very few places, particularly where the 
Xen headers used to include definitions which have since been removed. 
Last week a bug was raised about this, I'm not sure if this is what 
you're referring to:


https://www.redhat.com/archives/libvir-list/2007-December/msg00251.html

The greater point is that the headers which this patch includes are part 
of the public Xen API, and these header files are normally exported. 
Except that in both Debian and Fedora, because of the specific way that 
Debian and Fedora packaging call into a part of the Xen system of 
makefiles, they don't get exported.  Specifically:


  $(MAKE) -C $(SOURCE_DIR)/tools install [...]

Debian only do the 'make install' in a subdirectory.  If Debian did a 
normal 'make install', these headers would be installed, but there are 
other drawbacks to doing that, so exporting the headers by hand is the 
next best thing.


Rich.

--
Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/
Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.  Registered in
England and Wales under Company Registration No. 03798903


smime.p7s
Description: S/MIME Cryptographic Signature


Bug#402249: [PATCH] include necessary headers to build libvirt with xen support

2007-12-15 Thread Guido Guenther
Dear Debian Xen maintainers,
for your convenience I attach Richard's patch directly. It simply
installs the necessary headers so we're able to enable xen support in
libvirt for the default builds. Please apply.
Cheers,
 -- Guido

P.S.: building a separate libxen*-dev package (like Ubuntu does) looks
like a better long term solution
From 18915d87a7eebffe3264c43164aba82a345efb15 Mon Sep 17 00:00:00 2001
From: Guido Guenther [EMAIL PROTECTED]
Date: Sat, 1 Dec 2007 14:31:41 +0100
Subject: [PATCH] apply patch from Richard Jones to install the necessary headers to build libvirt with xen support (Closes: #402249)


diff --git a/debian/rules.real b/debian/rules.real
index e1c77e1..49c09ac 100644
--- a/debian/rules.real
+++ b/debian/rules.real
@@ -115,9 +115,15 @@ install-utils_$(ARCH): $(STAMPS_DIR)/build-utils_$(ARCH)
 	dh_clean -k
 	@rm -rf $(DIR)
 	$(MAKE) -C $(SOURCE_DIR)/tools install DESTDIR=$(CURDIR)/$(DIR) DISTDIR=$(CURDIR)/$(DIR) $(CONFIG)
+	mkdir -p $(CURDIR)/$(DIR)/usr/include/xen/io $(CURDIR)/$(DIR)/usr/include/xen/hvm $(CURDIR)/$(DIR)/usr/include/xen/arch-x86
+	install -m 644 $(SOURCE_DIR)/xen/include/public/io/*.h $(CURDIR)/$(DIR)/usr/include/xen/io
+	install -m 644 $(SOURCE_DIR)/xen/include/public/hvm/*.h $(CURDIR)/$(DIR)/usr/include/xen/hvm
+	install -m 644 $(SOURCE_DIR)/xen/include/public/arch-x86/*.h $(CURDIR)/$(DIR)/usr/include/xen/arch-x86
+	install -m 644 $(SOURCE_DIR)/xen/include/public/*.h $(CURDIR)/$(DIR)/usr/include/xen/
 	install -D -m644 debian/xen-utils.NEWS $(PACKAGE_DIR)/usr/share/doc/$(PACKAGE_NAME)/NEWS
 	install -D -m644 debian/xen-utils.README.Debian $(PACKAGE_DIR)/usr/share/doc/$(PACKAGE_NAME)/README.Debian
 	dh_install --sourcedir=$(DIR) usr/lib
+	dh_install --sourcedir=$(DIR) usr/include
 	dh_install --sourcedir=$(DIR) usr/share/xen-$(VERSION)$(ABINAME)
 	dh_pycentral
 	dh_strip