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.

Sign up for GitHub

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

Jump to bottom

remove MBINCompiler dependency #35

Closed
monkeyman192 opened this issue May 13, 2019 · 2 comments
Closed

remove MBINCompiler dependency #35

monkeyman192 opened this issue May 13, 2019 · 2 comments
Assignees
@monkeyman192
Milestone
post 1.0.0 release

Comments

@monkeyman192
Copy link
Owner

blocks #27
Since MBINCompiler is not cross-platform, it will be problematic to create any kind of automated test suites that can be run remotely.
There shouldn't be too many structs that need to be made to be exported.
TODO: List all the structs to add.
@monkeyman192 monkeyman192 self-assigned this May 13, 2019
@monkeyman192 monkeyman192 added this to the post 1.0.0 release milestone May 17, 2019
@monkeyman192
Copy link
Owner Author

Update: This may not be needed. If we can get the tests to be run on a container that has mono installed, we can build MBINCompiler for linux (current build profile is ubuntu.16.04-x64) and run using that which will be much nicer than completely removing the MBINCompiler dependency.

@monkeyman192
Copy link
Owner Author

I'm going to close this as a wont-do

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@monkeyman192 monkeyman192

Labels
None yet
Projects
None yet
Milestone
post 1.0.0 release
Development

No branches or pull requests
1 participant
@monkeyman192