This is mildly annoying to me:

$ darcs clone . remote:dir
works, but
$ darcs clone . remote:dir/
doesn't.

Allowing the latter makes darcs behave the same regardless of the target dir
being local or remote.

3 patches for repository http://darcs.net/screened:

patch 5da08695dc1ecb221a8bb6e9331ce4738d07a8f4
Author: Gian Piero Carrubba <gpiero@rm-rf.it>
Date:   Sun Dec 18 21:46:31 CET 2022
  * t/clone.sh: add some basic clone tests

patch 1316fe08bad1b5daa3857627a2b62685b3ddf37e
Author: Gian Piero Carrubba <gpiero@rm-rf.it>
Date:   Sun Dec 18 22:32:35 CET 2022
  * t/network/ssh.sh: also test clone to a remote target containing trailing slashes

patch 4f7e76c95d19efd74c8a8b565fadee9a4b8b9846
Author: Gian Piero Carrubba <gpiero@rm-rf.it>
Date:   Sun Dec 18 22:16:30 CET 2022
  * allow trailing slashes when cloning to a remote ssh target

Attachments

• patch-preview.txt
• t_clone_sh_-add-some-basic-clone-tests.dpatch

Amended bundle. The previous one failed in the case:

$ darcs clone . remote:/

This is still not permitted, but for other reasons independent from this 
patch. Moreover, the revised patch also fixes the case:

$ darcs clone . ssh://...

that was previously failing.

Attachments

• t_clone_sh_-add-some-basic-clone-tests.dpatch

Fixed tests. Is it ok to only send the amended patches, or should I 
resend the whole bundle?

Attachments

• t_network_ssh_sh_-also-test-clone-to-a-remote-target-containing-trailing-slashes.dpatch

It should be possible to apply the patches you intend by selecting some 
subset of the attached `.dpatch` files and applying everything in them.

I think that means you should resend the whole bundle in this case.

It's also helpful, at least for the tooling I use to process patches on 
the tracker, to use the edit link on the .dpatch files that are no 
longer needed and mark them as "dead". At least if you have permissions 
to.

I think the changes look good. Could you maybe add a comment to 
`sshCanonRepo` to describe what it is trying to achieve?

Updated bundle.

Attachments

• t_network_ssh_sh_-clone-to-a-remote-target-using-ssh___-uri-and-trailing-slash.dpatch

I have pushed your latest bundle to screened.