Bug#796784: Tentative patch for sending mails to signer

2016-07-12 Thread Chris Lamb
Raphael Hertzog wrote: > So I would also love to see it merged sooner rather than later. I've had a look at this and made a couple of improvements (see below). Can someone please check and then merge from the "send-mails-to-signer-796784" branch of https://github.com/lamby/dak? commit 3de2d3

Bug#796784: Tentative patch for sending mails to signer

2016-05-31 Thread Raphael Hertzog
Hi, On Tue, 31 May 2016, Salvatore Bonaccorso wrote: > On Wed, Dec 30, 2015 at 10:55:25PM +0100, Thorsten Alteholz wrote: > > I did some tests with the updated patch. I uploaded packages as > > maintainer/uploader or NMUs and accept- and reject-emails have been only > > sent to the configured emai

Bug#796784: Tentative patch for sending mails to signer

2016-05-31 Thread Salvatore Bonaccorso
Hi all, On Wed, Dec 30, 2015 at 10:55:25PM +0100, Thorsten Alteholz wrote: > Hi everybody, > > On Mon, 14 Dec 2015, Raphael Hertzog wrote: > >Thorsten, can you do some tests with the updated patch? > > I did some tests with the updated patch. I uploaded packages as > maintainer/uploader or NMUs

Bug#796784: Tentative patch for sending mails to signer

2015-12-30 Thread Thorsten Alteholz
Hi everybody, On Mon, 14 Dec 2015, Raphael Hertzog wrote: Thorsten, can you do some tests with the updated patch? I did some tests with the updated patch. I uploaded packages as maintainer/uploader or NMUs and accept- and reject-emails have been only sent to the configured email address (d..

Bug#796784: Tentative patch for sending mails to signer

2015-12-14 Thread Raphael Hertzog
Hi, On Sat, 12 Dec 2015, Ansgar Burchardt wrote: > > There's a new configuration entry DInstall::UploadMailRecipients that > > lists the recipients of the mails, either static ones (like > > d...@security.debian.org) or dynamic ones (like "maintainer", "changed_by" > > or "signer"). > > I prefer

Bug#796784: Tentative patch for sending mails to signer

2015-12-12 Thread Ansgar Burchardt
Raphael Hertzog writes: > I just created the attached patch. It's untested for now but I wanted you > to check if the approach used was matching your expectation. Yes, that's the idea I came up with some days ago. > There's a new configuration entry DInstall::UploadMailRecipients that > lists th

Bug#796784: Tentative patch for sending mails to signer

2015-12-01 Thread Raphael Hertzog
Hello Ansgar, I just created the attached patch. It's untested for now but I wanted you to check if the approach used was matching your expectation. There's a new configuration entry DInstall::UploadMailRecipients that lists the recipients of the mails, either static ones (like d...@security.debi