darcs

Issue 1167 Darcs transfer-mode is not versioned

Title Darcs transfer-mode is not versioned
Priority feature Status needs-implementation
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, exlevan, jaredj, jch
Assigned To
Topics ProbablyEasy, SSH

Created on 2008-10-25.10:19:56 by jch, last changed 2010-03-23.15:55:57 by kowey.

Messages
msg6447 (view) Author: jch Date: 2008-10-25.10:19:54
After discussion with Nicolas and Florent, we've found a few issues with
the ``darcs transfer-mode'' protocol.  I'm filing separate bugs for each
issue.

1. The protocol is not versioned.

This will lead to problems later on.  We recommend a new command

    OPTIONS list of options

to which the server replies

    OPTIONS list of options

Right now, the list of options is empty, so the server should be able to
parse an arbitrary line starting with ``OPTIONS'', and reply with just

    OPTIONS

                                        Juliusz
msg6451 (view) Author: kowey Date: 2008-10-25.11:33:38
I wonder if there is a way we can do this without introducing an extra
connection (one more reason to type password).

Perhaps options could be a command that darcs transfer-mode learns to parse?  We
will need to see how old darcs-transfer mode deals with this and how old darcs
client respond accordingly
msg6479 (view) Author: droundy Date: 2008-10-27.17:18:33
Better to make this backwards-compatible by making it

get options

so that the existing server can handle it just fine (by sending an
error back, indicating that it's version 1 of the protocol).

David
msg6480 (view) Author: kowey Date: 2008-10-27.17:21:40
I suppose we would have to name it something that is unlikely to actually appear
in anybody's repository, for example:

get darcs-transfer-mode-options
msg6482 (view) Author: droundy Date: 2008-10-27.17:33:06
That's not necessary, since the namespace isn't shared with files that
are in repositories.
msg6483 (view) Author: kowey Date: 2008-10-27.17:35:10
Excellent! Marking ProbablyEasy (especially since we need loads of easy jobs if
we're going to carry on sprinting in the years to come)
History
Date User Action Args
2008-10-25 10:19:56jchcreate
2008-10-25 11:33:40koweysetpriority: feature
nosy: jch, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
status: unread -> unknown
messages: + msg6451
title: Darcs transfer-mode is not versioned -> Darcs transfer-mode ought to be versioned
2008-10-27 17:18:35droundysetnosy: + droundy
messages: + msg6479
title: Darcs transfer-mode ought to be versioned -> Darcs transfer-mode is not versioned
2008-10-27 17:21:41koweysetnosy: droundy, jch, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
messages: + msg6480
2008-10-27 17:33:08droundysetnosy: droundy, jch, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
messages: + msg6482
2008-10-27 17:35:16koweysettopic: + ProbablyEasy, SSH
nosy: + jaredj
messages: + msg6483
2009-08-06 18:01:31adminsetnosy: + markstos, jast, Serware, darcs-devel, zooko, mornfall, tommy, beschmi, - droundy, jch, jaredj
2009-08-06 21:13:40adminsetnosy: - beschmi
2009-08-10 21:50:10adminsetnosy: + jch, jaredj, - tommy, markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:49:14adminsetnosy: - dagit
2009-08-19 12:42:46koweysetstatus: unknown -> needs-implementation
nosy: jch, kowey, simon, thorkilnaur, jaredj, dmitry.kurochkin
topic: + Target-2.4
2009-08-25 17:32:14adminsetnosy: + darcs-devel, - simon
2009-08-27 10:30:33koweysetnosy: jch, kowey, darcs-devel, thorkilnaur, jaredj, dmitry.kurochkin
2009-08-27 14:28:17adminsetnosy: jch, kowey, darcs-devel, thorkilnaur, jaredj, dmitry.kurochkin
2009-11-15 16:52:45tux_rockersettopic: - Target-2.4
2010-03-23 15:55:57koweysetnosy: + exlevan, - kowey, thorkilnaur