-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Errors opening a pull request: "Failed to nmap..." #1244
Comments
Hi @bwishan, we've had another report of this problem but have been unable to reproduce. Does this happen on any public repositories that we could access? |
I'll see if I can give it a try later this morning. @grokys |
@bwishan I've uploaded a build of GitHub for Visual Studio with a possible fix. Could you install it and let me know if it improves the situation? You can find it here: #1248 (comment) |
Hi, I just hit a similar case when I tried to open up a Pull Request to review the code. I attached ntsd, and the operation seems to be exhausting the heap. Note how Coincidentally, I work with @bwishan , so could it potentially be something with our GitHub enterprise installation? Before:
after:
|
@jcansdale Looks like the upgrade did the trick! Thanks for taking a look. |
@jcansdale , unfortunately, I can't even sign in with the new build. I see the login screen, and 'GitHub |
@machish Oh dear, it looks like you're having all the bad luck. Could you try following the steps and see if I can log in after that: #1242 (comment) It's encouraging to hear the new build worked for @bwishan! Hopefully once you manage to log it, it will work for you as well. Keep me posted. |
That workaround worked, and I was able to log in. (I had to restart VS for it to take effect). Thanks! |
@machish Thanks for the confirmation! It's true devenv and Resharper provide plenty of cover. 😉 We're working on a release that uses the latest stable version of LibGit2Sharp (v0.24): Because this update cuts across pretty much all of our functionality, it will need lots of testing. We're hoping to get something out by the end of the week. I'll see if I can get a new build to you before then. |
@machish I've added a link to the latest build here #1248 (comment) If you could give that a try and confirm whether (on not) the "Failed to nmap..." issue is also fixed is this version, that would be very much appreciated. 🤞 You should be able to simply open the .vsix and it should update to the new version. |
@jcansdale Yes, that build worked for both |
Environment:
GitHub Enterprise
Git repo has two remotes. One is a fork.
VS 2017 15.3.5
GitHub.VisualStudio 2.3.3.38
Repro:
Expected:
Opens PR
Actual:
Briefly spins and then displays the following error:
'Failed to nmap. No data written: Not enough storage is available to process this command.'
The text was updated successfully, but these errors were encountered: