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

Revert apt get integration #467

Merged

Conversation

ldennington
Copy link
Collaborator

#434 integrated git update-microsoft-git with apt-get. Since then, however, we've decided to move away from deploying to the Microsoft feeds and toward recommending installation from source (or the debian package published to our releases page). Reverting the associated commits accordingly.

@ldennington
Copy link
Collaborator Author

@derrickstolee If you don't mind taking a peek at commit 471453e, I'd appreciate it. It ended up being a fixup of a revert of a fixup with some modifications to the README.md file, so I want to make sure I didn't goof it up.

Copy link
Collaborator

@derrickstolee derrickstolee left a comment

Choose a reason for hiding this comment

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

This looks as clean as we could expect. I'll try to remember to further squash the Linux install instructions into the README commit during the next rebase.

Copy link
Member

@dscho dscho 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 a bit sad that Linux users cannot have the same convenience as macOS/Windows users, but it is what it is.

@ldennington
Copy link
Collaborator Author

It's a bit sad that Linux users cannot have the same convenience as macOS/Windows users, but it is what it is.

I hope to explore other homes for our packages in the future.

@ldennington ldennington merged commit c806769 into microsoft:vfs-2.34.0 Nov 16, 2021
ldennington added a commit to ldennington/git that referenced this pull request Jan 12, 2022
ldennington added a commit to ldennington/git that referenced this pull request Jan 19, 2022
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