-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Provide getopt for platforms that don't support it #3355
Comments
Did you consider to take the getopt stuff from musl? I think musl is a very neatly written libc implementation. It's also relatively modular and doesn't assume too much. It's also MIT-licensed so license shouldn't be a problem either. |
I've wrote the lib for the avr, I used some of musl's code and some public domain code plus my own code, so about the licence thing, is there a problem with that -using MIT and public domain code-, do I mention who's code did I use, even thought I did modify like half of it ? |
About MIT: there are several variants of the MIT license (most notably the Expat and the X11 license, both are compatible to (LGPLv2.1)[http://www.gnu.org/licenses/license-list.html]). If it is compatible to LGPLv2.1 it should be no problem to include them. Public Domain should be no problem either. |
thanks for the info about the licensing thing. |
Hi , Thanks |
@sudarshansrinivasan for avr-libc have a look at the comment by @N8Fear above. The implementation (the c-file) should go to @gebart how is the status for this on msp430 (+newlib?)? |
( |
I don't know about msp430 libc, but I guess newlib on msp430 has it if newlib on arm has it. |
I have to say since #6778 I'm pretty confused how newlib and msp430 work now ;-). But that's OT, I know. |
Hello.. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
Fixed by #9538 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
Fixed partly by #12457? |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
Is only avrlibc missing getopt? Now that avrlibc is actively maintained again, one could try to upstream getopt instead. Anyway, I think getopt is part of the POSIX environment that we can provide to ease life for developers new to RIOT. So I guess we should reopen this. |
newlib provides
getopt.h
which would implementing commands likeping6
far easier. Sadly msp430-libc and avr-libc do not provide this header. Conclusively, a lightweight implementation for those platforms would be much appreciated.The text was updated successfully, but these errors were encountered: