qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/3] -readconfig: accept fd=<fd> option (v2)


From: Eduardo Habkost
Subject: Re: [Qemu-devel] [PATCH 0/3] -readconfig: accept fd=<fd> option (v2)
Date: Mon, 19 Mar 2012 12:37:31 -0300
User-agent: Mutt/1.5.21 (2010-09-15)

On Mon, Mar 19, 2012 at 10:10:27AM -0500, Anthony Liguori wrote:
> On 03/19/2012 09:54 AM, Eduardo Habkost wrote:
> >This is a resubmit of a previous series I sent as a RFC, with some changes to
> >prepare for an upcoming patch that will make additional changes to the 
> >default
> >config-file loading code.
> >
> >This series needs be applied on top of the "./configure --confdir" series I
> >sent today.
> 
> Why not just use /dev/fd/N  ?

Personally, I don't like filenames with special meanings (as not every
OS has /dev/fd we would have to treat them specially), or filenames that
become non-extensible mini-languages by themselves. Many other
command-line options use the key=value syntax, and some already have an
"fd" option, so this just follows the convention.

Also, this is more extensible to allow more options to be added to
-readconfig if needed (for example: debugging options, or the
help=defconfig option I added on the RFC series I sent after this one).

-- 
Eduardo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]