-
-
Notifications
You must be signed in to change notification settings - Fork 303
Added "implement-libc" feature. #2564
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
base: master
Are you sure you want to change the base?
Conversation
This feature adds an option from project.json and build options to disable libc unreachable statements in order to be able to implement one for yourself. Useful feature for bare metal developing.
| macro CFile stderr() { return (CFile*)(uptr)STDERR_FD; } | ||
|
|
||
| module libc @if(!env::LIBC); | ||
| module libc @if(!env::LIBC && !env::IMPLEMENT_LIBC); |
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.
Rather than doing this, let's change this to env::NO_LIBC and then only set env::NO_LIBC is neither exists. Also look at the ones doing @if(env::LIBC) to see if they should be included when LIBC is implemented by the custom libc
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.
NO_LIBC is set by "link-libc", so are you suggesting to use that mechanism?
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.
I think he means changing the definition of NO_LIBC to something like const bool NO_LIBC = !LIBC && !CUSTOM_LIBC
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.
Yes exactly.
|
Rename |
|
Ping! |
I'm dealing with a heavy flu right now xD sorry for the delay, i'm unable to do anything atm |
|
It's fine, I hope you get well soon! |
This feature adds an option from project.json and build options to disable libc unreachable statements in order to be able to implement one for yourself. Useful feature for bare metal developing.