Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

installer: warn about 32-bit deprecation #482

Merged
merged 2 commits into from
Feb 22, 2023

Conversation

dscho
Copy link
Member

@dscho dscho commented Feb 21, 2023

As per git-for-windows/git#4279, Git for Windows is deprecating the 32-bit variant. Let's teach the installer to detect when a user is installing this variant and show an appropriate warning in that case. On a 32-bit Windows, they will see this type of dialog box:

image

On a 64-bit Windows, they will see this dialog box:

image

@dscho dscho self-assigned this Feb 21, 2023
Copy link
Member

@mjcheetham mjcheetham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks great! Don't suppose adding a link to info about the end of support is possible, given this is a simple MsgBox?

@dscho
Copy link
Member Author

dscho commented Feb 21, 2023

👍 Looks great! Don't suppose adding a link to info about the end of support is possible, given this is a simple MsgBox?

@mjcheetham unfortunately, it seems that this is not possible. The suggested workaround would be to let one of the buttons of the message box open a link, but out of the available buttons ("OK", "Cancel", "Abort", "Retry", "Ignore", "Yes", and "No"), nothing strikes me as an obvious good choice.

But I guess I could reword the message box thusly:

Git for Windows (32-bit) is nearing its end of support.
Would you like to open the ticket with more information?
Yes No

Another question is whether I should mention v2.40.x explicitly, or whether it is better to stay vague because we will continue to build 32-bit Git for Windows even with v2.41.x, it's just that the MSYS2 runtime won't be the latest and greatest anymore...

What do you think?

@mjcheetham
Copy link
Member

👍 Looks great! Don't suppose adding a link to info about the end of support is possible, given this is a simple MsgBox?

@mjcheetham unfortunately, it seems that this is not possible. The suggested workaround would be to let one of the buttons of the message box open a link, but out of the available buttons ("OK", "Cancel", "Abort", "Retry", "Ignore", "Yes", and "No"), nothing strikes me as an obvious good choice.

Yeah none of those make any sense.

But I guess I could reword the message box thusly:

Git for Windows (32-bit) is nearing its end of support.
Would you like to open the ticket with more information?
Yes No

That may work too (except the term "ticket" sounds weird..), but what about the 32-on-64 message? How about:

Git for Windows (32-bit) is nearing its end of support.
Read more information at gitforwindows.org/32bit.
OK

..and..

Git for Windows (32-bit) is nearing its end of support.
It is recommended to install the 64-bit variant of Git for Windows instead.
Read more information at gitforwindows.org/32bit.
Continue to install the 32-bit variant?
Yes No

Another question is whether I should mention v2.40.x explicitly, or whether it is better to stay vague because we will continue to build 32-bit Git for Windows even with v2.41.x, it's just that the MSYS2 runtime won't be the latest and greatest anymore...

What do you think?

Staying vague is probably better.

Since Git for Windows relies on the MSYS2 runtime for POSIX emulation as
required e.g. by the Bash (which is used by Git to execute aliases,
hooks, and the `git submodule` command), and since the MSYS2 project
already dropped support for the i686 variant of the MSYS2 runtime
(following Cygwin's lead), Git for Windows won't be able to provide full
support of 32-bit installers going forward.

Let's warn the users about this.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the deprecate-32-bit-Git-for-Windows branch from 7b778be to 5f9a91e Compare February 21, 2023 22:56
@dscho
Copy link
Member Author

dscho commented Feb 21, 2023

@mjcheetham I force-pushed an update (and with this update, this here PR needs git-for-windows/git-for-windows.github.io#43 to be merged first). The dialog boxes now look like this:

image

image

@mjcheetham
Copy link
Member

The dialog boxes now look like this

Looks great! I think adding the URL is helpful as overtime you can update the webpage with more concrete dates and version numbers etc.

The warning is intended to let users know that Git for Windows (i686
variant) is soon no longer going to be serviced.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the deprecate-32-bit-Git-for-Windows branch from 5f9a91e to c800a3f Compare February 22, 2023 07:56
@dscho
Copy link
Member Author

dscho commented Feb 22, 2023

Technically, it would be possible to implement a custom message box with an "Open link" button. However, the styling would be different, and to support the "Suppressible" part of SuppressibleMsgBox we would have to add even more code.

I've decided that there are more pressing things that I need to attend to ;-)

@dscho dscho merged commit 03fd35c into git-for-windows:main Feb 22, 2023
@dscho dscho deleted the deprecate-32-bit-Git-for-Windows branch February 22, 2023 08:13
@mjcheetham
Copy link
Member

Technically, it would be possible to implement a custom message box with an "Open link" button. However, the styling would be different, and to support the "Suppressible" part of SuppressibleMsgBox we would have to add even more code.

By "link" I mean adding the URL, which you have done. No need to add a hyperlink!

I've decided that there are more pressing things that I need to attend to ;-)

100%

@dscho
Copy link
Member Author

dscho commented Feb 24, 2023

Technically, it would be possible to implement a custom message box with an "Open link" button. However, the styling would be different, and to support the "Suppressible" part of SuppressibleMsgBox we would have to add even more code.

By "link" I mean adding the URL, which you have done. No need to add a hyperlink!

Still, a hyperlink would be nicer for users.

@asjimene
Copy link

Hello, please let me know if you would prefer an issue opened for this. It seems this change breaks installing the 32-bit version silently with /SUPPRESSMSGBOXES, as it defaults to "No" and cancels the installation when running silently.

Was this the intended outcome of this change?

@rimrul
Copy link
Member

rimrul commented May 31, 2023

It seems this change breaks installing the 32-bit version silently with /SUPPRESSMSGBOXES, as it defaults to "No" and cancels the installation when running silently.

Note that this behaviour only affects 64bit Windows, as the message box has no "No" option on 32bit Windows. See also #492 and git-for-windows/git#4347.

According to git-for-windows/git#4347 (comment) this is intentional.

@asjimene
Copy link

Ahh, that is what I needed to know. Thanks for pointing that out and apologies for not finding it on my own!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants