>> + if is_security_update and support != 'none': >> + if support == 'lts': >> + email_address = ['debian-lts@lists.debian.org'] >> + else: >> + email_address = ['t...@security.debian.org'] >> + listcc.extend(email_address) >> + > > I did not had much time recently, so I'm a bit late. I would be nice > here if that follows as well the intention of the file with the > support field, that is (beware only "pseudcode"): > > [...] > if is_security_update and support != 'none': > if support == 'lts': > email_address = ['debian-lts@lists.debian.org'] > elif support == 'security" > email_address = ['t...@security.debian.org'] > else > # nothing at at the moment, no more cases right now > [...] > > that is, in case we add another support value (you remember there was > once testing-security? ;-)) this can be done. > > I realize though you did now already upload the wheezy version, and > the unstable version as delayed, so guess we will need to leave it > now.
could you make it so that distributions.json contains directly the email address to use? do you really need the indirection of d.json -> support + support -> actual email (hardcoded in code)? -- Sandro "morph" Tosi My website: http://sandrotosi.me/ Me at Debian: http://wiki.debian.org/SandroTosi G+: https://plus.google.com/u/0/+SandroTosi
