This repository was archived by the owner on Jun 21, 2023. It is now read-only.
Use LibGit2Sharp v0.24.0 + fix crash when viewing PRs #1330
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Use the latest "stable" version of LibGit2Sharp.
Somehow putting the `repo` object in a `using` avoids the following issue: #1306
|
I haven't been able to this build crash using VS 2017 and the Avalonia solution. Went up and down the list of PRs and nothing. Please beat on it and see if you can make it crash! |
|
this is looking great to me. tested in both VS2015 and VS2017 with multiple repositories and haven't experienced a crash when viewing PRs. |
grokys
approved these changes
Nov 22, 2017
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The crash when viewing PRs was still happening in the GHfVS v2.3.5 bugfix release. This PR moves back to LibGit2Sharp v0.24.0 (because moving to v0.23.1 didn't fix it).
I've cherry-picked the fix from #1311. This wrapps the
Repositoryobject in ausingstatement. This appears to fix the issue, but I'm not convinced the fix is anything to do with disposing theRepositoryobject (it can crash before the object has been disposed).I'm not sure why the fix works, but it doesn't harm anything and it's worth getting out ASAP while we're working for a more concrete solution.
Fixes #1306