Ah, I get it. Thanks! I'll patch.
- Jamis
On Jan 12, 2007, at 10:08 AM, NeilW wrote:
>
> First patch the patch.
>
> Should be:
>
> wrap_length = (ENV['COLUMNS'] || 80).to_i - info[:longest] - 1
>
> The existing code fails for me on the standard 'spinner' and
> 'update_code' tasks both of which wrap the odd line by one character.
> I'm using standard Ubuntu Gnome terminals.
>
> Worth checking you have a genuine 80 column terminal first by
> generating a wrap, e.g.
>
> ruby -e '81.times {print "x"}'
>
> Rgs
>
> NeilW
>
>
>
>
> On Jan 12, 4:29 pm, Jamis Buck <[EMAIL PROTECTED]> wrote:
>> Thanks for the patch, Neil! However, I'm curious... I've never
>> noticed any problems. Can you give me an example where the existing
>> code breaks down?
>>
>> Regardless, I'll be applying your fix.
>>
>> - Jamis
>>
>> On Jan 12, 2007, at 9:03 AM, NeilW wrote:
>>
>>
>>
>>> That 'off by one' error in 'show tasks' finally annoyed me enough
>>> this
>>> afternoon to knock together a fix.
>>
>>> The problem is caused because there is a space separator in the
>>> 'puts'
>>> picture string which isn't taken into account by the line split
>>> algorithm.
>>
>>> Additionally I included a look in the environment for the value of
>>> 'COLUMNS', which on many Linux distributions is set to the width of
>>> the
>>> terminal. (Make sure you export it if you want it to work!)
>>
>>> Line 29 in 'standard.rb'
>>
>>> Change
>>
>>> wrap_length = 80 - info[:longest]
>>
>>> to
>>
>>> wrap_length = (ENV['COLUMNS'].to_i || 80) - info[:longest] - 1
>>
>>> Finally my task descriptions format correctly; harmony is
>>> restored :-)
>>
>>> NeilW
>
>
> >
--~--~---------~--~----~------------~-------~--~----~
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/capistrano
-~----------~----~----~----~------~----~------~--~---
