-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Visual Studio crashed in IncrementalAnalyzerProcessor #26171
Comments
Hi Thank you for reporting the issue? can you share logs in %temp%\servicehub\logs? looks like some setup is messed up. the callstack shows VS is trying to start out of proc process and that failed to be started. |
@jimmy9988 En-Shuo, have you reported this issue to FSharp team? it looks like somehow FSharp setup puts this in devenv.exe.config
but some VS drop doesn't have newtonsoft.json.dll in FSharp directory. causing everything that uses newtonsoft.json.dll in VS to fail. another thing is that, it points to version 10.0.0.0 but config has binding redirect to 9.0.0.0
making 2 different versions of newtonsoft.json.dll to be used. are these known issue? tagging @jinujoseph @KevinRansom @brettfo ... also, one of reliability bug @ivanbasov assigned to me (service hub failing to start) seems related to this. |
@nigurr can you check your directory (C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\CommonExtensions\Microsoft\FSharp\Newtonsoft.Json.dll) to see whether it has newtonsoft.json.dll ? |
here are service hub logs @nigurr shared. basically multiple services fail due to newtonsoft.json.dll issue. ... 04/17/2018 10:26:15 : Error : Error creating Microsoft.VisualStudio.ExtensionManager.ServiceModule.ExtensionManagerServiceModule instance of CLR service module 'extensionManager': System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. HResult='-2146232828' 04/17/2018 10:26:16 : Error : Error creating Microsoft.CodeAnalysis.Remote.RemoteHostService instance of CLR service module 'roslynRemoteHost': System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. HResult='-2146232828' |
@heejaechang I am using latest public preview (15.7 PR 3) and Newton.Json doesn't exist in In 15.6 Stable release, the Newton.Json exists in that folder |
cc @Pilchie for info |
@nigurr thank you ping @KevinRansom @brettfo according to @ivanbasov this is high priority
|
@brettfo deliberately removed that copy, so that F# would use the same version as everyone else, but it sounds like a the |
Can we get a VSTS bug on this for shiproom tracking ASAP please? |
I think the Newtonsoft issue is a red herring. I've been running 15.7 previews for weeks and none of them have a copy of newtonsoft.json present. The devenv.exe.config issue certainly needs to be fixed, but that would only cause issues at install time when everything is NGENed. Edit: |
I investigated this issue with @nigurr before. once we either copied newtonsoft.json to the FSharp folder or remove newtonsoft.json codebase to the FSharp folder, VS worked as expected. @jimmy9988 can you copy over newtonsoft.json to the FSharp folder and see whether VS still crash? |
I have a new theory as to what happened:
@jimmy9988 Can you try again with a clean install of VS and after deleting your hives under |
watson vso issue tracking this 600984 |
Fixed in dotnet/fsharp#4764. The 15.7 release of VS will contain the fix. |
Version Used: 15.7 Preview 3
Steps to Reproduce:
Stack trace:
I can share the repro machine/dump if required.
The text was updated successfully, but these errors were encountered: