On Tue, Mar 16, 2021 at 04:05:13PM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Mar 16, 2021 at 09:34:21AM -0700, Andi Kleen escreveu: > > > looks ok, but maybe make the option more related to CVS, like: > > > > > > --x-summary, --cvs-summary ...? > > > > Actually I don't think it should be a new option. I doubt > > anyone could parse the previous mess. So just make it default > > with -x > > In these cases I always fear that people are already parsing that mess > by considering the summary lines to be the ones not starting with > spaces, and now we go on and change it to be "better" by prefixing it > with "summary" and... break existing scripts.
I think it was just one version or so? FWIW perf has broken CSV output several times, I added workarounds to toplev every time. Having a broken version for a short time shouldn't be too bad. I actually had a workaround for this one, but it can parse either way. > > Can we do this with a new option? > > I.e. like --cvs-summary? If you do it I would add an option for the old broken format --i-want-broken-csv. But not require the option forever just to get sane output. Or maybe only a perf config option. -Andi
