Skip to content

Adjust the Windows download links #2001

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

Merged
merged 4 commits into from
Apr 30, 2025

Conversation

dscho
Copy link
Member

@dscho dscho commented Apr 30, 2025

Changes

  • Accommodate changes in Git for Windows when adjusting the download links.

Context

Git for Windows v2.47.1 was the first version to provide an ARM64 version, and v2.49.0 stopped providing an i686 version. This needs to be reflected in the download link handling.

I got inspired to make this change by git-for-windows/git-for-windows.github.io#61 of which this here PR is a natural companion.

@dennisameling would you mind testing (but not in Firefox, where this is known not to work) whether https://dscho.github.io/git-scm.com/downloads/win shows the ARM64 download link for you, i.e. not the following?

image

dscho added 2 commits April 30, 2025 10:36
As of v2.49.0, Git for Windows no longer ships 32-bit (also known as
"i686" or "Intel 32-bit") installers. However, Git for Windows started
shipping ARM64 versions since v2.47.1.

Let's accommodate for that by recording these artifacts, using the CPU
architecture rather than merely the bitness (which would now be
meaningless because both x64 and arm64 are 64-bit).

Subsequent commits will consume these new data items.

Signed-off-by: Johannes Schindelin <[email protected]>
Now that the `update-download-data.rb` script records x64/ARM64 data,
let's update `hugo.yml` after running this script.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho self-assigned this Apr 30, 2025
Copy link

@dennisameling dennisameling left a comment

Choose a reason for hiding this comment

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

It's still showing the x64 version for me, probably because of this error I found in the console:

image

image

const version = $('#auto-download-version');
const architecture = $('#auto-download-architecture');
const date = $('#auto-download-date');
if (link.length && version.length && bitness.length && date.length) {

Choose a reason for hiding this comment

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

Suggested change
if (link.length && version.length && bitness.length && date.length) {
if (link.length && version.length && browser.bitness.length && date.length) {

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, you're right, this must be the problem. But it should be changed to architecture instead of browser.bitness. Hold on a sec.

Copy link
Member Author

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

Can confirm that it now correctly shows all ARM64 links. Thanks! 🎉

image

Copy link
Member Author

Choose a reason for hiding this comment

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

@dennisameling could you have another look, please?

Choose a reason for hiding this comment

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

Not sure if you saw my comment from ~8 mins ago, but the ARM64 links are still showing up correctly as per your latest commit 👍🏼

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops. I had expected GitHub to update my browser page automatically, had not seen your answer! Thank you for confirming (twice!)!

dscho added 2 commits April 30, 2025 12:12
As of v2.47.1, Git for Windows provides an ARM64 version, and as of
v2.49.0 it stopped providing an i686 version.

The preceding commit updated the information in `hugo.yml` to reflect
this, and now it is time to use that information to present the download
links accordingly.

Signed-off-by: Johannes Schindelin <[email protected]>
The information is now stored by CPU architecture, no longer by bitness.
Let's drop the parts that are no longer needed (and wouldn't be updated,
anyway).

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the adjust-windows-download-links-again branch from 7a1b496 to d859fc7 Compare April 30, 2025 10:12
@dscho dscho merged commit 617d12c into git:gh-pages Apr 30, 2025
1 check passed
@dscho dscho deleted the adjust-windows-download-links-again branch April 30, 2025 10:32
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.

2 participants