[PATCH] samba-tool - make provision check for bind version
abartlet at samba.org
Thu Jan 9 03:00:05 MST 2014
On Wed, 2014-01-08 at 06:45 +0100, Jelmer Vernooij wrote:
> On Tue, Jan 07, 2014 at 04:08:17PM +1300, Andrew Bartlett wrote:
> > """
> > -
> > + from samba.provision import ProvisioningError
> ^ Please avoid non-toplevel imports. If this is to avoid a circular import, perhaps
> it makes sense to have a different kind of error object in this file?
Yes, this is to avoid a circular import. I would really appreciate
advise on other good options here - we really don't want more error
classes, certainly not one-off cases like a hypothetical
DNSProvisioningError, because then it will be even harder to map these
back to sensible errors in the samba-tool UI.
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical