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

Delete custom upgrader #1746

Merged
merged 2 commits into from
Jul 1, 2021
Merged

Conversation

derrickstolee
Copy link
Contributor

@derrickstolee derrickstolee commented Jun 30, 2021

Remove all code for the custom upgrader that ran the installer either from GitHub or from a NuGet feed.

This leaves gvfs upgrade non-functional, but a replacement will follow that calls winget upgrade vfsforgit.

This timer is responsible for checking if there is a new VFS for Git
version on a timer, then launches a notification to GVFS.Service.UI. I
was hoping to delete all of GVFS.Service.UI, but we need the toast
notifications for when we attempt to auto-mount and have a failure.

Signed-off-by: Derrick Stolee <[email protected]>
@derrickstolee derrickstolee changed the title Gvfs upgrade Delete custom upgrader Jun 30, 2021
@derrickstolee derrickstolee requested a review from mjcheetham June 30, 2021 20:07
@derrickstolee derrickstolee self-assigned this Jun 30, 2021
This leaves 'gvfs upgrade' non-functional

Signed-off-by: Derrick Stolee <[email protected]>
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.

🎉

@derrickstolee derrickstolee merged commit a452d79 into microsoft:master Jul 1, 2021
derrickstolee added a commit that referenced this pull request Jul 14, 2021
We removed the old upgrader in #1746 but delayed creating its replacement.

The plan was to use `winget upgrade`, but it is only experimental right now. Further, we would need to use `winget list` to determine if we needed `winget install` instead, and `winget list` is also experimental. The tool needs to mature more before we can create a dependency on it.

Instead, warn a user that tries to upgrade with `gvfs upgrade` that this CLI is no longer supported and instead point them to the repository's README which can help with the latest update.
derrickstolee added a commit that referenced this pull request Jul 15, 2021
Major Updates
----------------

* Comes with `microsoft/git` v2.32.0.vfs.0.4.
* This release will be available via `winget`.
* The `gvfs upgrade` verb is deprecated. Any future updates will be taken via `winget`.
* This version of Git has an upgrade mechanism that can advance independently of VFS for Git. Run `git update-microsoft-git` to update your Git version.
* The build system has been updated to use a more recent version of .NET.
* All macOS and POSIX code has been removed to simplify the build system.
* A FastFetch bug around deleting files has been fixed.
* A prefetch bug around downloading a small number of files has been fixed.

Special thanks to contributors @ldennington, @tyrielv, @50Wliu, and @SteveBenz.

Pull Requests
---------------

* #1738: Bumping version of update-winget action
* #1747: UpgradeVerb: write deprecation notice
* #1745: do not exit early if no blobs found for a period when prefetching
* #1740: Skip launching UI if running unattended
* #1737: Update Git to v2.32.0
* #1746: Delete custom upgrader
* #1741: Fix winget tag specification
* #1744: Overhaul build and project systems and drop Mac/POSIX code
* #1736: Update Git to v2.31.1.vfs.0.3
* #1735: Bumping winget action version
* #1734: Adding winget workflow
* #1733: Update Git to include 2.31.1
* #1730: Fix an issue with FastFetch when deleting files
* #1732: Use one NuGet feed
* #1726: Config: commitGraph.generationVersion=1
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