On Sun, Oct 14, 2007 at 10:48:28PM +0200, Luca Capello wrote:
> Hello!
>
> On Wed, 03 Oct 2007 02:53:21 +0200, Alberto Bertogli wrote:
> > On Mon, Oct 01, 2007 at 01:46:23PM +0200, Luca Capello wrote:
> >> Cc:ing upstream author, since this bug is clearly upstream and I'd
> >> like his opinion about my patch.
> >
> > In the future, please attach the patches uncompressed and inline. It
> > makes me able to comment on them inside the mail.
>
> Even if I don't like inline patches [1], the next time I'll do as you
> prefer.
Thanks. Using inline attachments is also fine by me.
> The main reason against the change is if /etc/darcsweb/ is intended to
> store other than a single file, config.py, which doesn't seem the
> case ATM.
I'm sorry, but I honestly don't believe this is worth breaking backwards
compatibility. If it's just to avoid creating a directory that in the
future might even be necessary or desirable, I don't think it's worth
it.
> I don't know anything about python, but what if the first place in
> sys.path contains a config.py? The problem should be the same :-(
The first place is always '.', that's why it works: it looks in the same
directory as darcsweb for a config file, and if it can't find one, it
looks in /etc/darcsweb.
That way you can have multiple darcswebs on the same machine with
different configurations, and avoid the name clashing with other python
modules.
Thanks,
Alberto
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
