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

Show the update message in the Action Center again #268

Merged
merged 1 commit into from
Nov 9, 2019

Conversation

dscho
Copy link
Member

@dscho dscho commented Nov 8, 2019

The git update-git-for-windows script tries to be smart and only show the Toast in Windows 8 and later. But the test was broken by an update to Cygwin v3.x, which changed the format of the output of uname -s.

This fixes that.

The format of `uname -s` changed in Cygwin v3.x, it seems, without
anyone noticing that this script tests for a trailing `-10.0` on Windows
10 (but it is now `-10.0-<build-number>`).

Let's fix that.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@rimrul
Copy link
Member

rimrul commented Nov 9, 2019

Yeah, that explains it. There's a typo in the PR description though. It should be Windows 8 and later.

@dscho
Copy link
Member Author

dscho commented Nov 9, 2019

There's a typo in the PR description though. It should be Windows 8 and later.

Whoops, that's not even a typo but a misunderstanding. Fixed.

@dscho dscho merged commit 75357fb into git-for-windows:master Nov 9, 2019
dscho added a commit that referenced this pull request Nov 9, 2019
The "Check daily for updates" feature [uses the Action Center
again](#268).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho deleted the fix-toast branch November 9, 2019 10:14
@HimbeersaftLP
Copy link

A window that displays the download progress would be useful, too. Especially for people with slower internet connections.

@dscho
Copy link
Member Author

dscho commented Dec 10, 2019

A window that displays the download progress would be useful, too. Especially for people with slower internet connections.

Feel free to work on this feature!

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.

3 participants