Bug#822463: perl: perlbug reports get rejected by exim4 due to long lines in 'perl -V'

Niko Tyni ntyni at debian.org
Thu Apr 28 20:32:32 UTC 2016


Control: forwarded -1 https://rt.perl.org/Ticket/Display.html?id=128020

On Thu, Apr 28, 2016 at 08:17:09PM +0100, Dominic Hargreaves wrote:
> On Thu, Apr 28, 2016 at 07:05:40PM +0300, Niko Tyni wrote:
> > On Tue, Apr 26, 2016 at 09:55:38PM +0300, Niko Tyni wrote:
> > 
> > > A better fix would be be to wrap everything, but I don't have a good patch
> > > for that. Hacking build_complete_message() could work I guess, but it's
> > > not used by all the sending backends (at least _send_message_mailsend();
> > > not sure we need to care about that.)
> > > 
> > > I think I need to look at this a bit more.
> > 
> > Revised set of two patches attached. Eyeballs would be welcome.
> 
> This looks sane to me. Thanks for taking the time to refactor!

Thank you for the review. I had to make one more modification to avoid
Text::Wrap eating empty lines from the body. Attached is the version I
sent upstream in [perl #128020].
-- 
Niko
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-perlbug-Refactor-duplicated-file-reading-code.patch
Type: text/x-diff
Size: 2650 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/perl-maintainers/attachments/20160428/1676bad7/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Make-perlbug-wrap-overly-long-lines.patch
Type: text/x-diff
Size: 2290 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/perl-maintainers/attachments/20160428/1676bad7/attachment-0001.patch>


More information about the Perl-maintainers mailing list