darcs

Issue 2462 --remote-darcs gets ignored for transfer-mode

Title --remote-darcs gets ignored for transfer-mode
Priority Status unknown
Milestone Resolved in
Superseder Nosy List bfrk
Assigned To
Topics

Created on 2015-07-02.12:27:38 by bfrk, last changed 2015-07-02.12:51:09 by bfrk.

Messages
msg18670 (view) Author: bfrk Date: 2015-07-02.12:27:37
To see this in action, do something with a remote (ssh) repo, specify
--remote-darcs /bla. Here is a real-life example:

> darcs pull
franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/BII-Controls/base-3-14-12
--remote-darcs /bla --debug
Beginning identifying repository .
Done identifying repository .
Identified darcs-2 repo:
/net/csr/srv/projects/ctl/franksen/epics/ioc/BII-Controls/head
Beginning identifying repository
franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/bii-controls/base-3-14-12
copySSH file:
franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/BII-Controls/base-3-14-12/_darcs/format
Disabling progress reports...
Starting new ssh connection to franksen@aragon
Detecting SSH settings
SSH settings: SshSettings {ssh = "ssh", scp = "scp -q", sftp = "sftp"}
ssh franksen@aragon darcs transfer-mode --repodir /opt/repositories
                    ^^^^^

Note the darcs command here (not /bla)

/controls/darcs/epics/ioc/BII-Controls/base-3-14-12/
grabSSH
src=franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/BII-Controls/base-3-14-12/_darcs/format
Reenabling progress reports.
Done identifying repository
franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/bii-controls/base-3-14-12
copySSH file:
franksen@aragon:/opt/repositories/controls/darcs/epics/ioc/BII-Controls/base-3-14-12/_darcs/prefs/sources

... and so on all working well because darcs just uses the default
remote darcs.

Fixing this would be very difficult. copySSH is called by things like
gzFetchFilePS where we don't have the remote darcs option available.

Not sure what to do here.

Perhaps eliminate --remote-darcs option and rely on the user@host's
shell setting to give us the correct darcs in the PATH? (As we do now
anyway for the transfer-mode).
msg18671 (view) Author: bfrk Date: 2015-07-02.12:51:08
PS: I hate it if people add features in this half-assed way. I mean, it
took me a few grep -r to see that DefaultRemoteDarcs is used in a number
of places, so it became obvious that the option isn't honored
consistently. Whoever added --remote-darcs must have been aware of this.

Very well, if it turns out to be too inconvenient to thread the
parameter through to the end, then please either abandon the effort
(it's not that the feature is essential), or else find a proper solution
(a global mutable variable might be justified here). But don't leave us
with half-working "features" that cannot be relied on to actually work
the way one expects.

Ok, I feel better now. Sorry for the rant. I guess the responsible party
has disappered long ago...
History
Date User Action Args
2015-07-02 12:27:38bfrkcreate
2015-07-02 12:51:09bfrksetmessages: + msg18671