[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