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

Generate C# SDK on Linux and Windows #159

Merged
merged 22 commits into from
Jun 19, 2018
Merged

Generate C# SDK on Linux and Windows #159

merged 22 commits into from
Jun 19, 2018

Conversation

jenow
Copy link
Contributor

@jenow jenow commented May 31, 2018

What does this PR do ?

Generate C# SDK for Linux and Windows, compatible Unity

How should this be manually tested?

Follow the instructions on the README

Note

Blocked by #162

@jenow jenow changed the title generate C# SDK Generate C# SDK May 31, 2018
@codecov-io
Copy link

codecov-io commented May 31, 2018

Codecov Report

Merging #159 into 1.x will decrease coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              1.x     #159      +/-   ##
==========================================
- Coverage   89.01%   88.93%   -0.09%     
==========================================
  Files         244      244              
  Lines        4407     4409       +2     
==========================================
- Hits         3923     3921       -2     
- Misses        452      456       +4     
  Partials       32       32
Impacted Files Coverage Δ
index/mDelete.go 90% <100%> (+1.11%) ⬆️
realtime/subscribe.go 71.42% <0%> (-5.72%) ⬇️
realtime/join.go 72.97% <0%> (-5.41%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf7fd06...c16b07b. Read the comment docs.

@jenow jenow added the wip label May 31, 2018
@jenow jenow changed the title Generate C# SDK Generate C# SDK on Linux and Windows Jun 5, 2018
@jenow jenow removed the blocked label Jun 5, 2018
@jenow jenow removed the wip label Jun 5, 2018
@@ -3,7 +3,7 @@ go:
- 1.9
dist: trusty
install:
- sudo apt-get install -y build-essential swig oracle-java8-installer g++ python3-dev
- sudo apt-get install -y build-essential swig oracle-java8-installer g++ python3-dev mono-mcs
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just for my understanding : the dependency on mono here is just required by swig, right?

Can you confirm the generated cs code can be consumed by any CLR implementation (regular.net, .net Core or mono)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No it cannot, the final step of the making of the dll is made by mono (mcs). If we want to to do this with regular .net or something else we have to change this.
I am not able to do this as I have absolutely no knowledge about .net and windows in general so I think we should do an issue if we want to include regular .net compiler and how to do it etc.
The goal of this PR is to get a draft of the C# SDK and then get feedbacks from people who knows it to then get a stable C# SDK

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, fine by me. I'll try to have a look at if the dll can be consumed by official .Net CLR (I doubt .Net core will) if I find time.

@jenow jenow added the blocked label Jun 13, 2018
@jenow jenow merged commit 78df949 into 1.x Jun 19, 2018
@jenow jenow deleted the swig-csharp branch June 19, 2018 12:24
@jenow jenow mentioned this pull request Sep 10, 2018
@jenow jenow mentioned this pull request Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants