Issue454

Title darcs + scp doesn't work for paths with whitespace
Priority bug Status in-progress
Superseder Nosy List beschmi, dagit, droundy, giksos, kowey, markstos, rgm, tommy
Assigned To Topics SSH

Created on 2007-05-10.02:43:50 by giksos, last changed 2008-05-14.08:34:04 by kowey.

Messages
msg4414 (view) Author: kowey Date: 2008-04-30.19:45:56
There was a patch for this, but it only handled the case for scp.  This seems
like it's worth fixing.  We just need the sftp case covered as well.
msg3276 (view) Author: markstos Date: 2008-02-09.18:54:27
It sounds like there is still uncertainty here about why this fails and if is
something we should even be addressing or whether it is "not-our-bug".
msg1817 (view) Author: kowey Date: 2007-07-12.20:49:41
Just wanted to note that we had applied this patch in stable, but then decided
to roll it back, because it wasn't worth the risk.  

I'd recommend that a revised patch which covers both the scp and sftp be submitted.

See message <20070530191915.GB217@dewdrop.local> on darcs-devel, or
http://thread.gmane.org/gmane.comp.version-control.darcs.devel/5830
msg1707 (view) Author: giksos Date: 2007-05-24.01:02:57
well, I guess the problem was that my postfix wasn't set up and running :) I
tried it once more, hope it arrives now.
msg1692 (view) Author: kowey Date: 2007-05-12.19:40:49
Hi,

On Sat, May 12, 2007 at 00:10:41 +0000, Misha Aizatulin wrote:
> ok, did it. Though I can't be quite sure that it is correct or that it is enough :)

For what it's worth, I haven't received anything from the mailing list.
Could you double-check?  The darcs send should result in a mail to the
darcs-devel@darcs.net list

Or if that doesn't work, maybe attach a darcs patch to the tracker by
sending it as an attachment.

Sorry to make you jump through hoops!  Just think of it as your first
contribution to the darcs community (pending review, of course).
msg1690 (view) Author: giksos Date: 2007-05-12.00:10:37
ok, did it. Though I can't be quite sure that it is correct or that it is enough :)
msg1686 (view) Author: kowey Date: 2007-05-10.18:09:52
Any chance you could darcs send a patch to
http://www.abridgegame.org/repos/darcs-stable/

(I'd say stable, as this seems innocuous enough a fix)
msg1680 (view) Author: giksos Date: 2007-05-10.02:43:41
Here is the problem:

====================
:~/tmp/darcs> mkdir two\ words
:~/tmp/darcs> cd two\ words/
:~/tmp/darcs/two words> darcs init
:~/tmp/darcs/two words> cd ..
:~/tmp/darcs> darcs get "localhost:tmp/darcs/two words"
Invalid repository:  localhost:tmp/darcs/two words

darcs failed:  (scp) failed to fetch: localhost:tmp/darcs/two words/_darcs/inventory
====================

Here is how I solved it for myself (a darcs patch):

===================
{
hunk ./External.hs 188
+                 tr ' ' = "\\ "
}
===================

I imagine that copySSHs might also need some change.

Browse related patches: unstable | stable

History
Date User Action Args
2008-05-14 08:34:04koweysetstatus: chatting -> in-progress
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, rgm, giksos
2008-04-30 19:45:57koweysetstatus: deferred -> chatting
nosy: + dagit
messages: + msg4414
2008-02-09 18:54:28markstossetstatus: chatting -> deferred
nosy: + markstos
messages: + msg3276
2007-07-23 20:57:54koweysettopic: + SSH
nosy: + rgm
2007-07-12 20:49:42koweysetnosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1817
2007-05-24 01:03:08giksossetnosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1707
2007-05-12 19:40:58koweysetnosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1692
2007-05-12 00:10:41giksossetnosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1690
2007-05-10 18:09:54koweysetstatus: unread -> chatting
nosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1686
2007-05-10 02:43:50giksoscreate