Re: svn commit: r244271 - head/usr.sbin/iostat

2012-12-17 Thread Robert Watson
On Sat, 15 Dec 2012, Adrian Chadd wrote: People may have scripts based on the current output. You should have introduced a new parameter which modifies the output behaviour in the way you wanted, rather than modifying the existing behaviour. Please consider reverting your change (which is

Re: svn commit: r244271 - head/usr.sbin/iostat

2012-12-16 Thread Mikolaj Golub
On Sat, Dec 15, 2012 at 05:00:39PM -0800, Adrian Chadd wrote: > Hi, > > People may have scripts based on the current output. > > You should have introduced a new parameter which modifies the output > behaviour in the way you wanted, rather than modifying the existing > behaviour. > > Please cons

Re: svn commit: r244271 - head/usr.sbin/iostat

2012-12-15 Thread Adrian Chadd
Hi, People may have scripts based on the current output. You should have introduced a new parameter which modifies the output behaviour in the way you wanted, rather than modifying the existing behaviour. Please consider reverting your change (which is a good idea, btw) and adding a new modifier

svn commit: r244271 - head/usr.sbin/iostat

2012-12-15 Thread Mikolaj Golub
Author: trociny Date: Sat Dec 15 18:21:09 2012 New Revision: 244271 URL: http://svnweb.freebsd.org/changeset/base/244271 Log: Change `iostat -Ix` to display total duration of transactions instead of average duration, and total busy time instead of %. This looks more useful when one runs `