On 2009/11/26 11:38:57, ced wrote: > OK for me
Works for me (Linux). Changelog missing. And would be nice to have some hint in README, that (now) client settings are version specific.
Ok http://codereview.appspot.com/161051/diff/8/2003 File doc/usage.rst (right): http://codereview.appspot.com/161051/diff/8/2003#newcode926 doc/usage.rst:926: Udo Spallek, Bertrand Chenal, Mattias Behrle, Anne Krings Really minor issue;), not related to this one: Mathias
http://codereview.appspot.com/161051/diff/8/2005 File tryton/config.py (right): http://codereview.appspot.com/161051/diff/8/2005#newcode17 tryton/config.py:17: if os.name == 'nt': Why not simply user os.path.expanduser('~') here? This does the same and handles some other cases, too (see e.g. std-module ntpath.py). Otherwise have a look at std-module user.py. http://codereview.appspot.com/161051/diff/8/2005#newcode19 tryton/config.py:19: return os.environ['HOME'] $HOME may not be set http://codereview.appspot.com/161051/diff/8/2005#newcode21 tryton/config.py:21: def get_config_dir(): duplicate code here. Shoudl read like: if os.name == 'nt': basedir = os.environ['APPDATA'] else: basedir = os.environ['HOME'] return os.path.join(basedir, '.config', 'tryton', VERSION ...) http://codereview.appspot.com/161051/diff/8/2005#newcode21 tryton/config.py:21: def get_config_dir(): Please add a short doc string here http://codereview.appspot.com/161051/diff/8/2005#newcode23 tryton/config.py:23: return os.path.join(os.environ['APPDATA'], '.config', 'tryton', std-module site.py uses: base = os.environ.get("APPDATA") or "~" http://codereview.appspot.com/161051/diff/8/2005#newcode25 tryton/config.py:25: return os.path.join(os.environ['HOME'], '.config', 'tryton', Again: $HOME may not be set.