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

Suggestion: Update documentation for new mpc #368

Closed
mikkoairaksinen opened this issue Jan 17, 2019 · 2 comments
Closed

Suggestion: Update documentation for new mpc #368

mikkoairaksinen opened this issue Jan 17, 2019 · 2 comments

Comments

@mikkoairaksinen
Copy link

While it is awesome that mpc should now work on Linux and MacOs, the documentation on how to make that happen is very unclear. It would be extremely beneficial to have an example project setup (or a written doc) detailing the (likely extremely common) use case of:

  1. .NET Core server project
  2. Unity project with subset of the code that is shared with the server project (and subset of that code is decorated with [MessagePackObject] etc
  3. an example .csproj somewhere in this project that can be used as the input for mpc
  4. a pre-build step configured into the server project that runs mpc (with detection of operating system to run the correct build of mpc)

I would be happy to contribute this documentation but frankly I don't know how to steps 3 and 4. Most attempts at 3 so far have either yielded an empty GeneratedResolver file or just not worked at all (when on MacOS)

@neuecc
Copy link
Member

neuecc commented Jan 22, 2019

You're right and thanks, welcome to contribute.
Current MacOS compatibility is slightly broken. #355
We should fix it but it is difficult.

@github-actions
Copy link

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days.

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

2 participants