OmniSharp/omnisharp-roslyn

System.NotSupportedException: Cake does not support /v2/gotodefinition

nils-a opened this issue · 0 comments

I raised this issue first in cake-build/bakery#137 - but I think it "belongs" here.

GoToDefinition in Cake files is no longer working (not sure since when..)

the log shows:

[dbug]: OmniSharp.Stdio.Host
        ************ Request ************
{
  "Type": "request",
  "Seq": 42,
  "Command": "/v2/gotodefinition",
  "Arguments": {
    "FileName": "d:\\_dev\\test\\Nils.Cake",
    "Line": 7,
    "Column": 23,
    "WantMetadata": true
  }
}
[fail]: OmniSharp.Stdio.Host
        ************  Response (0.5418ms) ************ 
{
  "Request_seq": 42,
  "Command": "/v2/gotodefinition",
  "Running": true,
  "Success": false,
  "Message": "\"System.NotSupportedException: Cake does not support /v2/gotodefinition\\r\\n   at OmniSharp.Endpoint.EndpointHandler`2.<HandleRequestForLanguage>d__20.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Host\\\\Endpoint\\\\EndpointHandler.cs:line 233\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Endpoint.EndpointHandler`2.<Process>d__16.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Host\\\\Endpoint\\\\EndpointHandler.cs:line 131\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Stdio.Host.<HandleRequest>d__14.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Stdio\\\\Host.cs:line 218\"",
  "Body": null,
  "Seq": 666,
  "Type": "response"
}

I'm guessing this came with https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.23.13

Digging a bit deeper I'm guessing it's a regression from #2168 : I think OmniSharp.Cake/**/GotoDefinitionHandler.cs should have been "modified" as well.