-
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
network: provide convenience functions #4418
Comments
I would have already used this several times |
@OlegHahm something like providing headers for the shell commands? |
Pretty much, yes. I think the Contiki folks have something like that in https://github.com/contiki-os/contiki/tree/master/apps |
AFAIK this issue is not completely solved, so I'll move it for the next release. |
Fixed title ;-) |
This issue is very broad. How about we compile a list of those functions and tackle them one-by-one? @OlegHahm I hope you don't mind but I'll start. |
I think this makes a lot of sense. Thanks! |
@kaspar030 did you? |
It seems to me back then I just edited the issue and added some. |
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. |
Currently, a lot of shell commands provide some convenient functions (mostly for networking stuff) like "printing interface information", "print the routing table", or "send a certain packet". I think it would be useful to factor out these functions to be used outside the shell. If, e.g., one wants to build an application without the shell but prints the configured addresses (like the microcoap example) or wants to send a simple link-layer packet, this could be useful. Otherwise either code from shell commands gets duplicated or shell commands are called directly without a documented and well-defined interface.
edit Fri 18. Nov 2016 kaspar: added initial list
Convenience functions we'd like to have (feel free to add):
The text was updated successfully, but these errors were encountered: