[MERGE] An SMTPConnection class for bzrlib
aaron.bentley at utoronto.ca
Wed Jun 20 01:43:40 BST 2007
-----BEGIN PGP SIGNED MESSAGE-----
Adeodato Simó wrote:
>> Responsibility for preventing a traceback should be taken at a higher
>> level, not here.
> Well, the problem is that, as I see it, the upper level can't really
> prevent the error from happening, because it can't know in advance what
> the SMTP server will react with an error code to.
I didn't mean to imply that the upper layer would prevent the error,
just the traceback.
> With this, I can't see how this is an internal error.
I tend to think that everything is an internal error until it's proven
to be user error.
> If it's an internal error, every call to send_email() has to catch
> SMTPError, and wrap it into a BzrCommandError or whatever appropriate;
*after properly determining that this is not some kind of internal error*
> Is my point clear (even if you don't agree)?
Your point is clear, and this appears to be something that reasonable
people may disagree on, so if you want to make it a non-internal error,
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----END PGP SIGNATURE-----
More information about the bazaar