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

Editor doesn't support projects using .csproj (project.json required) #51

Closed
mickaelistria opened this issue Jul 11, 2017 · 11 comments
Closed
Assignees

Comments

@mickaelistria
Copy link
Contributor

Currently, the editor, as it relies on Omnisharp-node-client suffering from OmniSharp/omnisharp-node-client#434 , is not able to provide edition assistance for dotnet 2.0 project (using .csproj description files).

@mickaelistria
Copy link
Contributor Author

@LucasBullen at the moment, I don't think we need to fix it right away as we don't have the needs nor the resource to follow bleeding edge development of dotnet-core. We'll get back to it when dotnet-core 2.0 is released.
The cool thing is that the wizards are already ready for dotnet-core 2.0 and are backward-compatible. That will be one less thing to implement later.

@mickaelistria
Copy link
Contributor Author

OmniSharp-node-client 7.0.7 can run with dotnet 2.0 but still isn't able to process .NET project using a .csproj file.

@AndrienkoAleksandr
Copy link

AndrienkoAleksandr commented Jul 26, 2017

OmniSharp/omnisharp-node-client#438 it should help for dotnet 2. I tested it with projects generated with help dotnet 2.

@mickaelistria
Copy link
Contributor Author

@AndrienkoAleksandr Thanks for the hint! I just tried it and it seems like the "initialize" request never returns and drives the client to timeout. Is this an issue you're aware of?

@AndrienkoAleksandr
Copy link

Hi, @mickaelistria . Do you have such problem fot dotnet 1.0.4 or 2.0 preview too? I used this with installed mono 5.2 beta http://www.mono-project.com/download/beta/. And second question Did you find some regression? Did it work earlier?

@mickaelistria
Copy link
Contributor Author

mickaelistria commented Jul 28, 2017 via email

@mickaelistria mickaelistria changed the title Editor doesn't support dotnet core 2.0 preview Editor doesn't support .csproj Jul 28, 2017
@mickaelistria mickaelistria changed the title Editor doesn't support .csproj Editor doesn't support projects using .csproj (project.json required) Jul 28, 2017
@AndrienkoAleksandr
Copy link

Hmm... I will see on it today.

mickaelistria added a commit that referenced this issue Jul 28, 2017
@AndrienkoAleksandr
Copy link

Hi @mickaelistria, I found that "--assembly-loader=strict" correct working only from 5.2 beta http://www.mono-project.com/docs/about-mono/releases/5.2.0/#strong-assembly-names and .csproj based projects working correct with this version. So I'am working on pull request where we check mono version and if it's 5.2 or higher we activate "--assembly-loader=strict", it should fix current breaking changes with mono 4.8 (I'm sorry for this breaking changes). It's sad but mono 4.8 can't working with this params... I asked guys from omnisharp-roslyn, they said in the some future release will be released binary with embedded mono...

@mickaelistria
Copy link
Contributor Author

mickaelistria commented Jul 31, 2017 via email

@AndrienkoAleksandr
Copy link

can you check please ?
OmniSharp/omnisharp-node-client#440

mickaelistria added a commit that referenced this issue Aug 2, 2017
Signed-off-by: Mickael Istria <[email protected]>
mickaelistria added a commit that referenced this issue Aug 9, 2017
Better support for dotnet-core 2.0 projects,
embeds its own version of mono

Signed-off-by: Mickael Istria <[email protected]>
mickaelistria added a commit that referenced this issue Aug 9, 2017
Better support for dotnet-core 2.0 projects,
embeds its own version of mono

Signed-off-by: Mickael Istria <[email protected]>
mickaelistria added a commit that referenced this issue Aug 9, 2017
Better support for dotnet-core 2.0 projects,
embeds its own version of mono

Signed-off-by: Mickael Istria <[email protected]>
@mickaelistria mickaelistria self-assigned this Aug 24, 2017
@mickaelistria
Copy link
Contributor Author

Fixed with mickaelistria@05eac6f

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

No branches or pull requests

2 participants