[netops] Re: svnmailer issue

[Andrej Zverev]

Well, if it's patch is okay, let's ask portmrg to approve it and update
checksum.


André Malo пишет:
> * Tom McLaughlin wrote: 
>
>   
>> I don't use svnmailer and it's a little late here but is the attached
>> patch correct?
>>     
>
> Nope ;-)
> The "unicode" string is the type of the config parameter, which should not 
> be changed. 8bit is one of the values.
>
> I've attached another patch, which should change the default value to 8bit.
>
>   
>> In addition, the 1.0.6 tarball was rerolled.  Anyone have a copy of the
>> old one that can be compared to the new one to see what changes have
>> been made?  Maybe this secondary problem can wait until after freeze
>> since we have a copy of the distfile cached on the FBSD mirrors.
>>     
>
> That would be me. I had to reroll the archived versions after a harddisk 
> crash some months ago (only the current version was backupped). There's no 
> code change between them. Only some date strings in the docs and stuff like 
> that were changed.
>
> nd
>   
> ------------------------------------------------------------------------
>
> diff -Nur svnmailer-1.0.6~/src/lib/svnmailer/notifier/_textmail.py svnmailer-1.0.6/src/lib/svnmailer/notifier/_textmail.py
> --- svnmailer-1.0.6~/src/lib/svnmailer/notifier/_textmail.py	2006-02-04 20:22:10.000000000 +0100
> +++ svnmailer-1.0.6/src/lib/svnmailer/notifier/_textmail.py	2006-10-17 09:29:49.000000000 +0200
> @@ -137,7 +137,7 @@
>              u"base 64" : "B",
>              u"8bit"    : "8",
>              u"8 bit"   : "8",
> -        }.get((self.getTransferEncoding() or u'').lower(), 'Q'))
> +        }.get((self.getTransferEncoding() or u'').lower(), '8'))
>  
>          for mail in mails:
>              mail.update(headers)
>