Re: [pcp] pcp updates: 'collectl2pcp' importer

[Nathan Scott <nathans@xxxxxxxxxx>]

----- Original Message -----
> On 05/16/2013 10:36 AM, Nathan Scott wrote:
> > Looking good.  Some small stuff from reviewing...
> 
> thanks for that

No worries mate.

> > - the -a option seems a bit wierd in collectl2pcp.c - a mandatory
> > option?  I was expecting syntax more like:
> > $ collect2pcp infile outfile
> 
> how about: collectl2pcp [-v] [-D N] output input [input ...]
> or perhaps collectl2pcp [-v] [-D N] input [input ...] output

The latter syntax matches the name of the command "collectl -> pcp"
so I'd recommend that.

> I'll send out another update with these changes, probably late tonight.

OK; be handy to do the QA test(s) first - since it all works already -
and then you'll be able to check for regressions from all these little
tweaks.  *hint, hint*  ;-)

cheers.

--
Nathan