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/
Home Products Support Community News


[Xen-devel] Re: [PATCH] ioemu: support non-default install directory

To: Christoph Egger <Christoph.Egger@xxxxxxx>
Subject: [Xen-devel] Re: [PATCH] ioemu: support non-default install directory
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Thu, 31 Mar 2011 18:20:56 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 31 Mar 2011 10:21:28 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D8C71EA.9040700@xxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4D8C71EA.9040700@xxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Christoph Egger writes ("[PATCH] ioemu: support non-default install directory"):
> This patch allows to install qemu-dm into a non-default directory
> like everything else in the tools.

Thanks but:

>  qemu.1: qemu-doc.texi
> -     $(SRC_PATH)/texi2pod.pl $< qemu.pod
> +     perl $(SRC_PATH)/texi2pod.pl $< qemu.pod
>       pod2man --section=1 --center=" " --release=" " qemu.pod > $@

This isn't covered by the commit message.

Please would you be sure to review your commits before you send them
to check that there aren't these kind of surprises in them.  If you
aren't able to do that then we have to do it for you, which means your
submissions go to the bottom of the pile ...


Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>