On Mon, 2008-05-19 at 17:55 +0100, Dr J A Gow wrote: > On Sat, 2008-05-17 at 14:51 +0100, Mark Ellis wrote: > > > I'll throw in my two penneth for a laugh :) > > > > I've always considered copying the default file to ~/.synce excessive. > > In fact I would consider the _application_ copying the file anywhere to > > be excessive, that is a distro or an install thing. How about this... > > > > 1) sensible defaults in the app (which they are) > > > > 2) (maybe) check for a config file in /etc, if it doesn't exist, fine > > > > 3) check for a config file in ~/.synce, if it doesn't exist, fine > > > > You then get defaults, or system config, or user config, but it's all > > done knowingly ie. if someone changes or places or removes a config > > file, they know it's been done. > > > > If we need a template config.xml, stick it in /usr/share/doc/examples. > > > > This seems quite a sensible change, except that I am not sure about > checking /etc (sync-engine is a user-based app and config could well be > different for each user, e.g. file sync paths). > > I honestly can't remember why I put the file copy in there in the first > place but on reflection it does seem daft. I'll change it in a day or > so. > > John. > >
Agree with the /etc thing, it doesn't really fit. The copy is reasonable in the context of putting the config file somewhere as a template, since looking in the python tree isn't the first thing most people would think of. I've recently come across some ideas on installing this kind of thing outside of the tree with setuptools (in the Vodafone Mobile Connect Card software out of interest). I'll try and get around to doing something with this, should make access to the template a bit easier. Mark ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ SynCE-Devel mailing list SynCE-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/synce-devel
