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

CoreCLR support #34

Closed
LennyPenny opened this issue Nov 13, 2015 · 21 comments
Closed

CoreCLR support #34

LennyPenny opened this issue Nov 13, 2015 · 21 comments

Comments

@LennyPenny
Copy link

Would be awesome if you could compile the nuget package for the coreclr framework.
Can I do that myself somehow for now?

@vtortola
Copy link
Owner

Yes, give it a try and we can see how difficult would be. There are some dependencies on other assemblies, so I not completely sure it would work out of the box.

@LennyPenny
Copy link
Author

I unfortunately have no idea how to compile c# packages myself and installing them. Any idea where I could read that up?

@vtortola
Copy link
Owner

Neither do i :D

@pehadavid
Copy link

hey,

dnx support will be definitely great ! I can do it, but it will require large modifications (including in the sln structure).
Here's the main approach :

  • changing every csproj to xproj (with just dnx451 support)
  • checking that the classic dotnet platform support is not broken
  • adding dnxcore 5.0 support

Tell me what you think

@vtortola
Copy link
Owner

Would that somehow break compatibility with VS2013 and .NET 4.5.1 ?

@pehadavid
Copy link

I don't know if VS2013 support xproj project. But it will not break .net 4.5.1 support.

@vtortola
Copy link
Owner

I am going to be incredibly busy until Xmas. I can create a new branch and we can experiment there, however if you need my support I cannot guarantee that will happen straight away :( But definitely sounds interesting and I want to at least try 👍

Would that suit you?

@pehadavid
Copy link

An experimental branch would be nice ;)
Thanks !

@pehadavid
Copy link

I have good news. I tried to implement dotnet core support, and it went well.
you can check it on my forked repo. (here : https://github.com/papci/WebSocketListener).

If you want to check if it's compatible with VS2013, feel free to do it :)

Thanks.

@StefH
Copy link
Contributor

StefH commented Jan 10, 2016

I've forked from papci and added support for more frameworks:

  • net45, net451 and net452
  • dnx451 and dnx451
  • dnxcore5

And i was able to compile dnxcore5 on Linux, however the UnitTest project cannot compile/run because Moq is not present for dnxcore5. (Maybe also nunitlite has some issues.)

@vtortola
Copy link
Owner

I still have to take a look to the changes that @Papci did (sorry, i had busy Xmas :( )

@StefH do you have the code somewhere?

@StefH
Copy link
Contributor

StefH commented Jan 11, 2016

Code is present in my branch (which was forked from papci)

https://github.com/StefH/WebSocketListener

@vtortola
Copy link
Owner

Sorry I have been trying to find free time to take a look at this and due some personal committments I just can't :( I see you guys are doing progress. I will try to follow up and in the meantime please reach me for any question you may have about the standard code. Thanks!

@StefH
Copy link
Contributor

StefH commented May 20, 2016

dotnet rc2 support --> netstandard 1.3
See pull request:
#65

@Supergeek
Copy link

Considering the difficulty of importing DLLs into Core, it would be greatly appreciated if someone would publish one of their Core-compatible forks to Nuget.

@vtortola
Copy link
Owner

vtortola commented Dec 6, 2016

Hi, I think @StefH already did some work on this https://www.nuget.org/packages/vtortola.WebSocketListener.dnx/2.2.0.1-beta-00002

@StefH
Copy link
Contributor

StefH commented Dec 6, 2016

This is some time ago. I'll see if I can find some time to check my project and upgrade if needed. And also check that NuGet package.

@StefH
Copy link
Contributor

StefH commented Dec 10, 2016

I've updated the project again to latest netstandard.

Initial NuGets are created and pushed to myget.org : https://www.myget.org/F/websocketlistener/api/v3/index.json

Please try.

I'll create a PR later (I've tried to keep the changes/differences minimal, but I'll have some discussions with @vtortola on that I guess...)

@deandob
Copy link

deandob commented Jan 15, 2017

Hi, do post when these dotnet core changes are merged back in the main branch and available. Thanks for the time spent on this.

@StefH
Copy link
Contributor

StefH commented Feb 19, 2017

Code is merged to main. However some work is needed. See also #93

@vtortola
Copy link
Owner

Done un 3.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants