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

Re: [Xen-devel] [PATCH] Fix xenmon/xentrace compile on Solaris



For the Xentrace side, I'd say:

> On Tue, Aug 07, 2007 at 02:48:38PM +0100, Keir Fraser wrote:
> >   1. Don't use argp at all in xentrace/xenmon -- do they really need
> > super-advanced argument parsing such that getopt will not suffice? As
> > low-level not-usually-user-visible daemons I would say that porting them
> > to getopt would be acceptable even if the command-line option syntax
> > needs changing a little to cleanly do so.

This is what I was going to suggest.  The existing syntax should probably work 
fine, argp just expresses it more nicely (IMO) at the expense of non-GNU 
portability.

> We'd presumed that somebody had gone out of their way to use argp for a
> reason. If it's not necessary we're certainly fine with doing that instead.

Nope, no reason :-)

As far as I'm concerned, feel free to drop argp and replace with getopt.  
Maintaining existing syntax might be nice, but isn't required.

As far as Xenmon is concerned, I didn't know it used argp.  The original 
authors might have some useful comment to make, but my guess would be that 
they used argp to match the existing practice in Xentrace.

Cheers,
Mark

-- 
Dave: Just a question. What use is a unicyle with no seat?  And no pedals!
Mark: To answer a question with a question: What use is a skateboard?
Dave: Skateboards have wheels.
Mark: My wheel has a wheel!

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.