| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Wed, 1 Dec 2010 15:55:04 -0200 From: Arnaldo Carvalho de Melo <acme@...stprotocols.net> To: eranian@...il.com Cc: Stephane Eranian <eranian@...gle.com>, linux-kernel@...r.kernel.org, peterz@...radead.org, mingo@...e.hu, paulus@...ba.org, davem@...emloft.net, fweisbec@...il.com, perfmon2-devel@...ts.sf.net, robert.richter@....com Subject: Re: [PATCH] perf: add csv-style output to perf stat Em Wed, Dec 01, 2010 at 06:24:46PM +0100, stephane eranian escreveu: > On Wed, Dec 1, 2010 at 6:06 PM, Arnaldo Carvalho de Melo <acme@...stprotocols.net> wrote: > > Spaces are being added, gack, will fix. Tried to use the same option letter and > > long option name as in 'sort': > > > > -t, --field-separator=SEP > > use SEP instead of non-blank to blank transition > > > > But then 'perf stat' already uses -t for --tid, so in 'stat' we would have to > > use '-x'/--field-separator. > > > Fine with me. > I can re-spin the patch to add the flexibility to name your SEP. please > > Argh, I think we should stop using short options, only assigning something when > > it gets from seldomly used to just before making it the default 8-) > > I tend to use the short options.... If you're ok with keeping --field-separator as the long option to match what is in 'perf report', deal :-) - Arnaldo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists