-
Notifications
You must be signed in to change notification settings - Fork 1.3k
add support for USER_C_MODULES #1754
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
Conversation
Why do you want this? We will not promise that our internal C APIs are stable. The only way to have something native that is maintained is by merging it into this repo. |
@tannewt I followed the original MicroPython discussion on this over months. The idea is just to make it lower overhead to add special-purpose modules, segregated in another directory. Anyone using this would need to maintain a fork. I don't think we'd recommend using this except for private stuff. It doesn't flow with @dmazzella Anything general-purpose (e.g., an FFT module, or something like that), we would like to see submitted to us as a PR, even if it's turned off by default. |
One advantage of having this in our fork is that it will make merging from upstream easier next time, since it's already included in MPy. At the next merge, we'd either have to merge or discard it (manually). |
@dmazzella Please describe what you are using this for. |
Feel free to comment on why you need this and I'll reopen. I'll close for now. |
@tannewt sorry if I answer with so much delay. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bringing this in.
No description provided.