From: "brian m. carlson" <sandals@crustytoothpaste.net>
To: Junio C Hamano <gitster@pobox.com>
Cc: Aditya Garg <gargaditya08@live.com>,
git@vger.kernel.org, Eric Sunshine <sunshine@sunshineco.com>
Subject: Re: [PATCH] help: inform about 'git update-git-for-windows' on Windows
Date: Wed, 21 May 2025 23:00:51 +0000 [thread overview]
Message-ID: <aC5bI_Y9-ZQPXJ4t@tapette.crustytoothpaste.net> (raw)
In-Reply-To: <xmqq5xht8uxm.fsf@gitster.g>
[-- Attachment #1: Type: text/plain, Size: 2192 bytes --]
On 2025-05-21 at 22:23:33, Junio C Hamano wrote:
> "brian m. carlson" <sandals@crustytoothpaste.net> writes:
> > I don't think this belongs in our codebase. It should instead be
> > carried as a patch in Git for Windows. The reason is that there are a
> > variety of possible projects that compile for Windows—Git for Windows,
> > Cygwin, MINGW, etc.—and only one of them ships this binary. It is even
> > possible for users to compile their own Windows binaries, which I know
> > is at least done by Microsoft as well as some Git contributors on
> > Windows.
> >
> > This change might be misleading or incorrect as it might tell users to
> > invoke a binary which is not present or to update software in a way
> > which is not via the normal package mechanism. For instance, telling a
> > MINGW or Cygwin user to run that command would not result in anything
> > useful or desired happening.
>
> Do you mean that this is OK if the #ifdef were more specific to
> Git-for-Windows? Just being curious.
I don't think that would be a good idea, either. There's no such #ifdef
to my knowledge and we have lots of ways for people to update software.
We don't tell people to run commands to update to a newer version of
their Debian package because that's a responsibility of the packager or
distributor, and so the same policy applies here. If Debian wants that
message to be included, then they can apply a patch and receive any bug
reports or other feedback related to that message; same goes for Git for
Windows.
I also happen to know that in some corporate environments proxy problems
cause the updater to break (which is not in any way a surprise) and
there are also cases where antivirus false positives flag the updater or
other tools. We do not in any way want to receive reports about those
problems or the updater and if we avoid recommending it, then we aren't
responsible for it. Otherwise, we'll inevitably get a request to allow
people to configure that message because it doesn't work in their very
special corporate environment and they don't want to confuse their
users.
--
brian m. carlson (they/them)
Toronto, Ontario, CA
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 325 bytes --]
next prev parent reply other threads:[~2025-05-21 23:00 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-20 19:31 Proposal: Add tip to use git update-git-for-windows when running git help or git version on Windows Aditya Garg
2025-05-21 14:22 ` [PATCH] help: inform about 'git update-git-for-windows' " Aditya Garg
2025-05-21 21:57 ` brian m. carlson
2025-05-21 22:23 ` Junio C Hamano
2025-05-21 23:00 ` brian m. carlson [this message]
2025-05-22 3:18 ` Aditya Garg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aC5bI_Y9-ZQPXJ4t@tapette.crustytoothpaste.net \
--to=sandals@crustytoothpaste.net \
--cc=gargaditya08@live.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=sunshine@sunshineco.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox