Andrew Fresh wrote on Mon, Jul 18, 2011 at 06:13:36PM -0700: > 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. [...] > I think this should actually fix adjust_columns. I looks correct, and it does fix the issue. Committed, thanks! > 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;