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

make the .net library work with .net core #42

Closed
irmen opened this issue Oct 10, 2016 · 9 comments
Closed

make the .net library work with .net core #42

irmen opened this issue Oct 10, 2016 · 9 comments

Comments

@irmen
Copy link
Owner

irmen commented Oct 10, 2016

No description provided.

@ststeiger
Copy link

Mobius depends on this libary.
If it isn't .net core ready, then Mobius can't be ported to .NET Core.
https://github.com/Microsoft/Mobius
When Mobius can't be ported to .NET Core, Apache Spark-SQL can't be used in .NET Core.

@irmen
Copy link
Owner Author

irmen commented Nov 16, 2017

Actually, I think we should focus on the .NET Standard to target... https://docs.microsoft.com/en-us/dotnet/standard/net-standard
Target should likely be .NET STandard v1.1 or v1.2

@ststeiger
Copy link

ststeiger commented Nov 17, 2017

Yes, this is correct.
However, I'd focus on Netstandard 2.0 instead.
Otherwise, you'll have to make too many (pointless) changes.
Nothing against NetStandard 1.1/1.2 however, if you have the time for that.
But there's no point in starting a project.json project, when this has already been deprecated.

@irmen
Copy link
Owner Author

irmen commented Nov 17, 2017

When targeting NetStandard 2.0 I'll have problems with building it with Mono on my Linux box: Debian stretch, which is shipped with mono 4.6. So NetStandard 1.6 is the highest, and I really wonder what shall have to be changed when aiming for 1.2? I guess I'll find out once I get around to this.
I suppose I have to use Visual Studio for this

@ststeiger
Copy link

No, actually JetBrains Rider will work fine on Linux.
Just need to install the .NET Core SDK on the machine.

https://www.microsoft.com/net/learn/get-started/linuxubuntu
sudo apt-get update
sudo apt-get install dotnet-sdk-2.0.2

Other distros - choose dropdown on site.
Well, with NetStandard 1.6 you stand a chance, since it contains reflection.
But if you need, for example [Serializable], then you require .NET-Core 2.0+

See https://github.com/davidfowl/NetStandard/blob/master/platforms.cs

As shown here
https://docs.microsoft.com/en-us/dotnet/standard/net-standard
for NetStandard 2.0, you'll need mono 5.4, if you want to do it with mono (which isn't required), while you probably have mono 4.6.

@irmen
Copy link
Owner Author

irmen commented Jan 24, 2018

Seems Rider creates .net standard 2.0 solutions, so that will likely be the target

@ststeiger
Copy link

I'd also go with 2.0.
Seems to be the most reasonable at the moment.
Lower will die out anyway.

@irmen
Copy link
Owner Author

irmen commented Jan 24, 2018

migrated to dotnet core and netstandard 2
moved to xunit, a few tests still fail but the rest seems to be working fine already

@irmen
Copy link
Owner Author

irmen commented Jan 26, 2018

Fixed all tests and published a 4.20 release to nuget.org @ststeiger could you have a look if this version works for you?
Thanks for pointing Rider out to me btw, it's a very nice tool

@irmen irmen closed this as completed Jan 26, 2018
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