Loading...

mailman-developers@python.org

[Prev] Thread [Next]  |  [Prev] Date [Next]

Re: [Mailman-Developers] Probe messages should not sent Precedenceheader Aamir Khan Sat Feb 18 10:01:33 2012

Mark Sapiro <[EMAIL PROTECTED]> wrote:

>
> Which means that bin/disabled.py won't work because it hasn't been
> fully converted from 2.1 yet. It should be using the send_probe()
> function instead of sendNextNotification().
>

Yes. I also think so. I have changed the function to send_probe() and ran
through all the test again to make sure that nothing breaks.


>
> There should be two places. bin/disabled.py should be calling
> send_probe() instead of the non-existent sendNextNotification() list
> method.
>
> Also, send_probe() should be called when delivery is initially disabled
> as a result of a call to IBounceProcessor(), but I am unable to follow
> how this works or even if it is implemented yet at least in part
> because I haven't yet learned anything about how things like
> zope.component.getUtility() work.
>

I am also not able to find out the relevant code, as i am completely new to
the flow of how mailman works. Maybe, barry can help me on this one.

>
>
> The patch itself looks good except for style. We normally don't put
> whitespace around the = in keyword=value arguments in definitions and
> calls. PEP 8 <http://www.python.org/dev/peps/pep-0008/> says "Don't
> use spaces around the '=' sign when used to indicate a keyword
> argument or a default parameter value."
>
> I will keep this in mind. I have created another branch with partial
changes suggested above. Here, you can find the changes,

http://bazaar.launchpad.net/~syst3mw0rm/mailman/808821/revision/7080



-- 
Aamir Khan | 3rd Year  | Computer Science & Engineering | IIT Roorkee
_______________________________________________
Mailman-Developers mailing list
[EMAIL PROTECTED]
http://mail.python.org/mailman/listinfo/mailman-developers
Mailman FAQ: http://wiki.list.org/x/AgA3
Searchable Archives: 
http://www.mail-archive.com/mailman-developers%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-developers/alexiscircle%40gmail.com

Security Policy: http://wiki.list.org/x/QIA9