[darcs-devel] darcs patch: correction for send.sh test (and 2 more)

Juliusz Chroboczek Juliusz.Chroboczek at pps.jussieu.fr
Wed Dec 21 06:04:47 PST 2005


> Oops! I can fix that and resubmit the whole absolute path thing as
> one integrated patch if you want

No, separate patches are easier.

> And what I added was to translate any parameters starting with slash
> to be as if the user had typed them in relative to the original_wd : 

Just to be clear: I'm not claiming what you're doing is incorrect.
I'm just trying to convince myself it doesn't break in some cases.

In particular, I think that some commands take paths relative to the
repository directory, not to cwd.  So I worry that you might end up
incorrectly fixing a remote path.

I'd feel more relaxed if you had individual commands do their
normalisation, rather than have the normalisation done at such a high
level.

> Of course, if there is some elegant alternate solution which doesn't
> involve all this silly path manipulation, I'm all for it :-)

Well, you're trying to work around the issue of having multiple name
spaces on a single system.  The obvious fix is to dump Cygwin, but I'm
afraid Zooko will get angry if I mention that.

                                        Juliusz




More information about the darcs-devel mailing list