Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the SVN-Notify CPAN distribution.

Report information
The Basics
Id: 68706
Status: resolved
Priority: 0/
Queue: SVN-Notify
People
Owner: Nobody in particular
Requestors: GNUSTAVO [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 2.79
Fixed in: 2.83

History Show all quoted text
  Wed Jun 08 12:13:10 2011 GNUSTAVO [...] cpan.org - Ticket created
Subject: Implement option --smtp-port in svnnotify
I need to make svnnotify send e-mail via SMTP to a server listening on port 465. I couldn't find a way to specify this via the command line. So, I made the attached patch to SVN/Notify.pm version 2.79. (I've seen that issue #40188 implements the same thing in its proposed patch. However, that patch deals with other things in order to make it possible to use SMTP over SSL. Which, incidentally, is exactly what I need to do. The SSL part I was able to do making SVN::Notify talk with a stunnel daemon which intermediates its conversation with Google's SMTP servers. But I needed it to talk with local port 465. Hence this smaller patch.)
  Sun Jun 12 23:07:14 2011 dwheeler [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #68706] Implement option --smtp-port in svnnotify
Date: Sun, 12 Jun 2011 20:07:05 -0700
To: bug-SVN-Notify [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
On Jun 8, 2011, at 9:13 AM, Gustavo Leite de Mendonça Chaves via RT wrote: Show quoted text
> I need to make svnnotify send e-mail via SMTP to a server listening on port 465. I couldn't find a way to specify this via > the command line. So, I made the attached patch to SVN/Notify.pm version 2.79.
Hrm. The patch doesn't seem to have gone through. BTW, you'll want to create the patch against SVN, here: https://svn.kineticode.com/SVN-Notify/trunk/ Best, David
  Sun Jun 12 23:07:15 2011 The RT System itself - Status changed from 'new' to 'open'
  Sun Jun 26 01:03:44 2011 dwheeler [...] cpan.org - Correspondence added
On Sun Jun 12 23:07:14 2011, DWHEELER wrote: Show quoted text
> BTW, you'll want to create the patch against SVN, here: > > https://svn.kineticode.com/SVN-Notify/trunk/
And now I've moved it here: https://github.com/theory/svn-notify/ Got a patch? Best, David
  Sun Jun 26 01:03:45 2011 dwheeler [...] cpan.org - Status changed from 'open' to 'stalled'
  Tue Jun 28 13:53:09 2011 GNUSTAVO [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #68706] Implement option --smtp-port in svnnotify
Date: Sun, 26 Jun 2011 10:32:37 -0300
To: bug-SVN-Notify [...] rt.cpan.org
From: Gustavo Leite de Mendonça Chaves <gnustavo [...] cpan.org>
2011/6/26 David Wheeler via RT <bug-SVN-Notify@rt.cpan.org> Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=68706 > > > On Sun Jun 12 23:07:14 2011, DWHEELER wrote: >
> > BTW, you'll want to create the patch against SVN, here: > > > > https://svn.kineticode.com/SVN-Notify/trunk/
> > And now I've moved it here: > > https://github.com/theory/svn-notify/ > > Got a patch? >
I'm so sorry for the delay. The patch is going attached to this message. Thanks. -- Gustavo.

Message body is not shown because sender requested not to inline it.

  Tue Jun 28 13:53:12 2011 The RT System itself - Status changed from 'stalled' to 'open'
  Wed Jun 29 00:05:02 2011 david [...] kineticode.com - Correspondence added
Subject: Re: [rt.cpan.org #68706] Implement option --smtp-port in svnnotify
Date: Tue, 28 Jun 2011 21:04:53 -0700
To: bug-SVN-Notify [...] rt.cpan.org
From: "David E. Wheeler" <david [...] kineticode.com>
On Jun 28, 2011, at 10:53 AM, Gustavo Leite de Mendonça Chaves via RT wrote: Show quoted text
> I'm so sorry for the delay. The patch is going attached to this message.
Applied, thanks! https://github.com/theory/svn-notify/commit/3466784a7dc8171292ccfe1d5d23b4e3fe0baf54 Best, David
  Wed Jun 29 00:05:34 2011 dwheeler [...] cpan.org - Status changed from 'open' to 'patched'
  Tue Aug 13 09:54:28 2013 dwheeler [...] cpan.org - Status changed from 'patched' to 'resolved'
  Tue Aug 13 09:54:28 2013 dwheeler [...] cpan.org - Fixed in 2.83 added