On Mon, Jul 18, 2011 at 12:32:02AM -0700, Daniel C. Sinclair wrote: > The day after installing the latest snapshot I got the usual > /usr/libexec/security email - but it appeared to be double spaced for > the setuid/device changes sections. It is actually just trailing > spaces on each line, which made them wrap on an 80 column terminal. > > This gets rid of them (and reduces the initial email by over 7KB):
yay! Sorry about that, my fault, probably should have noticed. > Someone more willing to muck with perl may want to fix the > 'adjust_columns' routine where the spaces get injected. I think this should actually fix adjust_columns. l8rZ, -- andrew - http://afresh1.com I think I understand, but my stubborn brain refuses to admit it until I beat it into submission by proof upon proof. -- Michael Shiloh <michaelshiloh1...@gmail.com> Index: security =================================================================== RCS file: /cvs/src/libexec/security/security,v retrieving revision 1.15 diff -N -u -p -u security --- security 20 Jun 2011 21:53:53 -0000 1.15 +++ security 19 Jul 2011 01:10:57 -0000 @@ -594,6 +594,7 @@ sub adjust_columns { if (!$s[$_] || length $row->[$_] > $s[$_]); } } + $s[-1] = ''; my $fmt = join ' ', map "%-${_}s", @s; return map { sprintf $fmt, @$_ } @table;
