-
Notifications
You must be signed in to change notification settings - Fork 105
AOT support #50
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
Comments
Sorry I've never used Enums.NET from an AOT app. Can you explain what things are broken in Enums.NET in an AOT environment? Is there some reflection I'm doing that's not supported? |
It looks like I tried addressing AOT support for Unity in #23 but never received feedback if it fixed it. |
I'm not an expert of AOT. But as I understand, if you use anything about reflection, then it's not AOT compatible. Please check this issue: nissl-lab/npoi#1156 It's not so urgent as I don't believe AOT can get popular in a short time. But you can check if it's possible to make Enums.NET AOT compatible. |
@karakasa Would you be able to enlighten me on what changes or attributes I should add to add AOT support? Thanks. |
The basic steps are as follows:
Most of step 3 will likely involve annotating generic type parameters, such as: https://github.com/Singulink/Singulink.Enums/blob/cb3845ea1d2bf4f2d090c44dd77a13668ef7def1/Source/Singulink.Enums/EnumExtensions.cs#L24, but you can get more info on what may be required here: @tonyqus Shameless plug for the lib linked above, just released v2 with AOT support if you want something similar to enums.net that is AOT compatible right now. |
Hi, I'm the guy who maintains NPOI. Enums.NET is really nice. Thank you for the nice work.
I'd like to check if you have plan to support AOT because someone suggests me to get rid of Enums.NET in order to support AOT. But I don't like his idea. I think it's better to check with you first.
The text was updated successfully, but these errors were encountered: