Re: [Libvir] [PATCH] Another Report error in virsh.c code.

2008-03-25 Thread S.Sakamoto
Would you give me a comment on my opinion ? If not, I make a patch that is suitable for my opinion. I forgot to attach a patch... I attach it. If not, please apply it. Shigeki Sakamoto. virsh.patch Description: Binary data -- Libvir-list mailing list Libvir-list@redhat.com

Re: [Libvir] PATCH: Add SCSI HBA backend for storage driver

2008-03-25 Thread Daniel Veillard
On Mon, Mar 24, 2008 at 11:15:47PM +, Daniel P. Berrange wrote: And this time with the patch included... [...] PARTED_REQUIRED=1.8.0 +HAL_REQUIRED=0.5.0 With HAL being in Solaris now, maybe that could even be made non-linux specific [...] @@ -584,6 +585,8 @@ AC_ARG_WITH(storage-iscsi,

Re: [Libvir] [PATCH] reindent __virErrorMsg with spaces instead of tabs

2008-03-25 Thread Guido Günther
HI Daniel, On Mon, Mar 24, 2008 at 03:56:07AM -0400, Daniel Veillard wrote: Hum, could you send those kind of patches as attachments in the future ? Basically assuming spaces/tabs are correctly preserved in an email body is taking unecessary risks. Also having a name for the patch and being

Re: [Libvir] [PATCH] Another Report error in virsh.c code.

2008-03-25 Thread Richard W.M. Jones
On Tue, Mar 25, 2008 at 03:22:41PM +0900, S.Sakamoto wrote: Would you give me a comment on my opinion ? If not, I make a patch that is suitable for my opinion. I forgot to attach a patch... I attach it. If not, please apply it. Shigeki Sakamoto. OK, I've committed your final version.

Re: [Libvir] [PATCH] reindent __virErrorMsg with spaces instead of tabs

2008-03-25 Thread Daniel Veillard
On Tue, Mar 25, 2008 at 11:06:02AM +0100, Guido Günther wrote: HI Daniel, Hi, [...] Now for the content of the patch, it seems to remove all tabs and replace them with 8 spaces. Is that really a better way, why use 8 characters when you can use one for the same purpose ? One could

Re: [Libvir] Patch for routed virtual networks

2008-03-25 Thread Mads Chr. Olesen
man, 24 03 2008 kl. 19:00 +, skrev Daniel P. Berrange: On Mon, Mar 24, 2008 at 10:52:41AM +0100, Mads Chr. Olesen wrote: Anything further I can do to help get this patch commited? I have been running with it, without problems across restarts, etc., for a couple of weeks now. I

[Libvir] error : could not connect to xen:///; error: failed to connect to the hypervisor

2008-03-25 Thread Felix Krohn
I can't find the reason for the following error: # virsh libvir: error : could not connect to xen:/// error: failed to connect to the hypervisor # virsh -c ///var/lib/xend/xend-socket libvir: error : could not connect to ///var/lib/xend/xend-socket error: failed to connect to the hypervisor #

Re: [Libvir] PATCH: Add SCSI HBA backend for storage driver

2008-03-25 Thread Daniel P. Berrange
On Tue, Mar 25, 2008 at 03:24:48AM -0400, Daniel Veillard wrote: On Mon, Mar 24, 2008 at 11:15:47PM +, Daniel P. Berrange wrote: And this time with the patch included... [...] PARTED_REQUIRED=1.8.0 +HAL_REQUIRED=0.5.0 With HAL being in Solaris now, maybe that could even be made