| |
|
 |
|
 |
|
|
|
| |
|
|
xen-devel
Re: [Xen-devel] [PATCH] ioemu: make daemonize optional
|
To: |
"Daniel P. Berrange" <berrange@xxxxxxxxxx> |
|
Subject: |
Re: [Xen-devel] [PATCH] ioemu: make daemonize optional |
|
From: |
"Stephen C. Tweedie" <sct@xxxxxxxxxx> |
|
Date: |
Fri, 08 Feb 2008 12:52:54 +0000 |
|
Cc: |
"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Samuel Thibault <samuel.thibault@xxxxxxxxxxxxx> |
|
Delivery-date: |
Fri, 08 Feb 2008 04:53:27 -0800 |
|
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxx |
|
In-reply-to: |
<20080207143754.GE30230@xxxxxxxxxx> |
|
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
|
List-unsubscribe: |
<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
|
Organization: |
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 |
|
References: |
<20080207142251.GF9343@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20080207143754.GE30230@xxxxxxxxxx> |
|
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
Hi,
On Thu, 2008-02-07 at 14:37 +0000, Daniel P. Berrange wrote:
> On Thu, Feb 07, 2008 at 02:22:51PM +0000, Samuel Thibault wrote:
> > ioemu: make daemonize optional
>
> Why ? Littering up the source code with lots of #ifdef isn't
> very nice for maintainance. Can we at least have some explanation
> of the actual need for these changes along with the patch.
Seconded --- when going through patch history in mercurial, it is often
hugely helpful to have the rationale in the patch commit log, rather
than having to dig through email archives to get it.
--Stephen
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
| |
|
Home