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;

Reply via email to