On Friday 15 November 2013 15:18:26 thierry bordaz wrote:
1- I'd replace the following:
-from lib389._constants import DN_DM
+from lib389 import DN_DM
Files starting with _ are "private" and are just facilities for the developer.
We could consider to rename _constants to constants
2- To isolate testing tools from the library I would put
args_standalone & co in a proper bugfix harness file/class that we can add to
lib389, eg:
from lib389.bugfix import args_standalone
Otherwise people/projects using lib389 will be full of our testing code.
Peace,
R.
--
Roberto Polli
Community Manager
Babel S.r.l. -
http://www.babel.it
T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680
P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma)
CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere
confidenziale per i destinatari in indirizzo.
E' vietato l'inoltro non autorizzato a destinatari diversi da quelli indicati
nel messaggio originale.
Se ricevuto per errore, l'uso del contenuto e' proibito; si prega di
comunicarlo al mittente e cancellarlo immediatamente.