darcs

Patch 1633 Resolve issue2545: prepend "--" to non-option arguments in ssh invocation

Title Resolve issue2545: prepend "--" to non-option arguments in ssh invocation
Superseder Nosy List gpiero
Related Issues
Status needs-screening Assigned To
Milestone

Created on 2018-01-06.07:17:30 by gpiero, last changed 2018-01-08.19:29:20 by gpiero.

Files
File name Status Uploaded Type Edit Remove
improve-debug-messages-about-invocation-of-external-ssh-commands.dpatch gpiero, 2018-01-08.19:25:51 application/octet-stream
issue2545_command-execution-via-ssh-uri.sh-portability-fix.dpatch gpiero, 2018-01-08.19:29:20 application/octet-stream
tests_sshlib_-add-a-helper-function-for-initializing-a-remote-repo.dpatch gpiero, 2018-01-06.07:17:29 application/octet-stream
See mailing list archives for discussion on individual patches.
Messages
msg19817 (view) Author: gpiero Date: 2018-01-06.07:17:29
patch 315bc9fb58ce4245be4bb443d29ae3d1fccd2257 is named '* Resolve
issue2545:
...' in order to automatically close the bug (and because it effectively
solve
the issue:).
Anyway I plan to send a more comprehensive patch that will not allow
host names
starting with a '-'. That is the same approach used by git, svn and,
IIRC, hg.
The rationale is that the use of '--' for signaling end of options is
implementation-dependent (well, so is the use of '-' as the first char of an
option). Given that AFAIK a host name starting with '-' is invalid as per
RFC-1123, I don't see any problems in implementing it also in darcs.

At last, git also disabled repositories starting with a '-'. While not
the best
choice for naming a directory (at least on POSIX), I think (but not
thoroughly
checked) that such a dir name would not create problems in darcs and I'm not
leaning to support this restriction.
Attachments
msg19819 (view) Author: gpiero Date: 2018-01-08.19:25:51
patch 4ecf394c343d80f68c25d53bc1acf2eed1bd0a27
Author: Gian Piero Carrubba <gpiero@rm-rf.it>
Date:   Mon Jan  8 13:57:46 CET 2018
  * Fix test broken by patch 8706d6e70c1aa572f383882c371a9bafbec3dc38
  (Improve debug messages about invocation of external SSH commands)
Attachments
msg19820 (view) Author: gpiero Date: 2018-01-08.19:29:20
patch 1b543dc9c0e2ca4ced934e58eed081c3f69c5627
Author: Gian Piero Carrubba <gpiero@rm-rf.it>
Date:   Mon Jan  8 20:12:29 CET 2018
  * t/n/issue2545_command-execution-via-ssh-uri.sh: portability fix
Attachments
History
Date User Action Args
2018-01-06 07:17:30gpierocreate
2018-01-08 19:25:51gpierosetfiles: + improve-debug-messages-about-invocation-of-external-ssh-commands.dpatch
messages: + msg19819
2018-01-08 19:29:20gpierosetfiles: + issue2545_command-execution-via-ssh-uri.sh-portability-fix.dpatch
messages: + msg19820