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

.Net Standard 2.0 version #149

Open
melucas opened this issue Feb 8, 2018 · 6 comments
Open

.Net Standard 2.0 version #149

melucas opened this issue Feb 8, 2018 · 6 comments

Comments

@melucas
Copy link

melucas commented Feb 8, 2018

Any thoughts on converting this project to .Net Standard 2.0 or .Net Core now that MS has depreciated the portable projects?

@rdavisau
Copy link
Owner

rdavisau commented Mar 1, 2018

Yes, I do have plans to switch it over in the near future. That said, I had expected it would be possible to use sockets directly from netstandard2, though I hadn't verified that.

@melucas
Copy link
Author

melucas commented Mar 7, 2018

So far it a is working as expect. However, the complier moans about back converting the assembly to .Net 4.x. I have tried it in a xamarin forms app running as Universal Windows and on an iPhone and my UDP connection are working and returning data.

@alanu86
Copy link

alanu86 commented May 2, 2018

are there any new information about porting to netstandard2.0 ?

@newdeal30
Copy link

That feature would be absolutly great!

@dog44wgm
Copy link

It was all working beautifully for me until I tried to reference the Socket property in Release mode.

// Should not be called on a TcpSocketClient that is not already connected.
if (m_oClient.Socket.Connected)
    await m_oClient.DisconnectAsync();

when I got this

warning : ILTransform : warning ILT0003: Method 'Communicator.d__10.MoveNext()' will always throw an exception due to the missing method 'TcpSocketClient.get_Socket()' in assembly 'Sockets.Plugin'.

so it can build and deploy fine, but then yes, it throws.
The problem doesn't exist in Debug.

@dog44wgm
Copy link

Sorry, I forgot to mention that the problem exists in UWP only.

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

5 participants