Re: tortoiseplink ssh variant still needed?

2 messages, 2 authors, 2021-10-25 · open the first message on its own page

Re: tortoiseplink ssh variant still needed?

From: Junio C Hamano <hidden>
Date: 2021-10-23 21:53:37

Sven Strickroth [off-list ref] writes:
IIRC tortoiseplink was initially created by TortoiseCVS which is
nowadays not maintained any more 
(<http://www.tortoisecvs.org/download.shtml>).
In TortoiseSVN and TortoiseGit tortoiseplink was modified six years
ago (around 2015-03-07) to ignore the "--batch" parameter and also
accept "-p" additionally to "-P"  (cf. 
<https://osdn.net/projects/tortoisesvn/scm/svn/commits/26348>).

Therefore, my first question: Does Git still want to support very old
versions of tortoiseplink or should I provide a patch which drops 
support for it?
You sent the question to me so I am answering, but this depends on
(1) who the users who use TortoisePlink are and (2) by dropping
support for it, what we would gain.

I am guessing that nobody other than those on Windows use
TortoisePlink, and that everybody other than those who build their
own Git from source use Dscho's Git-for-Windows, and I further
assume that the GfW comes with its own copy of OpenSSH.

So our intended audience is those who started using Git on Windows
back when TortoisePlink was still a thing, are still happily using
TortoisePlink, and are willing to only update Git but not migrate to
OpenSSH.  How big that audience is, I do not know, as I do not do
Windows.

How much maintenance burden is the "support" costing us?  A quick
scan in connect.c tells me that the "add --batch to the command
line" would be the only thing we would be able to shed; everything
else seems to be shared with plink and putty.

So...?

Re: tortoiseplink ssh variant still needed?

From: Johannes Schindelin <hidden>
Date: 2021-10-25 13:04:48

Hi Junio,

On Sat, 23 Oct 2021, Junio C Hamano wrote:
Sven Strickroth [off-list ref] writes:
quoted
Therefore, my first question: Does Git still want to support very old
versions of tortoiseplink or should I provide a patch which drops
support for it?
[...]

I am guessing that nobody other than those on Windows use
TortoisePlink, and that everybody other than those who build their
own Git from source use Dscho's Git-for-Windows, and I further
assume that the GfW comes with its own copy of OpenSSH.

So our intended audience is those who started using Git on Windows
back when TortoisePlink was still a thing, are still happily using
TortoisePlink, and are willing to only update Git but not migrate to
OpenSSH.  How big that audience is, I do not know, as I do not do
Windows.
Nobody really knows, but we started discouraging `plink` usage (also
`tortoiseplink` usage) already way back in the msysGit days. AFAIR we
simply ran into too much trouble, and started to only offer `plink` as an
option if the user had _any_ PuTTY saved sessions.
How much maintenance burden is the "support" costing us?  A quick
scan in connect.c tells me that the "add --batch to the command
line" would be the only thing we would be able to shed; everything
else seems to be shared with plink and putty.
Since I have to assume a very small usership, I would think that we can
drop support for the older `tortoiseplink`. But you're right, what does it
_buy_ us?

My guess is that Sven wants to go further and enable the `-o SetEnv` thing
for protocol v2 (which we figured out together, over in the Git for
Windows bug tracker, to be turned off when pushing). But that would
require the `tortoisegitplink` variant, I think.

So maybe a better idea would be to focus on introducing support for
`tortoisegitplink` and work on the `-o SetEnv` issue, and leave the
`--batch` code alone for now.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help