-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
fix #2103 - allow unsafe code #2104
fix #2103 - allow unsafe code #2104
Conversation
@keveleigh, good question. i'll give it a try |
verified that with BOTH files present, Unity successfully compiles the dll with IL2CPP. |
Updated PR description to match the current contents. |
Do we also need to make sure this file gets included into the asset package? What happens if someone has a file like this already declared? |
I will be adding these files to the asset package for the May release. If the user has one, the package importer should show it as already there (and not override) |
that's not exactly how it works. It'll only override if the meta file id is the same. |
We had been shipping .rsp files in earlier releases. Is the concern that ours may overwrite an existing rsp? |
no, mostly if they collide, but then again, if people are using this file they most likely know how to handle conflicts with it. |
Add csc.* to fix Unity not enabling Allow Unsafe Code in the generated projects when building for the .NET Backend.
Tested on Unity versions:
Fixes: #2103