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
gcoap: move dependencies to actual module #7071
Conversation
cf283cc
to
893fcbb
Compare
Thanks for the improvement, @kYc0o. I agree that any use of gcoap will need the nanocoap pkg and the gnrc_sock_udp module for the foreseeable future. To make this work complete, please make the same changes for a couple of other make files:
Also, the posix module no longer is required, thanks to #6897, which addressed #5959. If you are comfortable with eliding the posix module in the migration, it works for me. I tested with native and samr21-xpro. |
+1 for this PR. @kYc0o any progress? |
Sorry, I did't have internet this week-end, I'll work on it ASAP. @kb2ma thanks for your suggestions! I'll include them in this PR. |
893fcbb
to
bc2387e
Compare
Amended and force-pushed. |
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.
ACK
By using gcoap a bit I noticed that the actual dependencies for gcoap were declared for the example rather than for the actual module.
As far as I can see, the example doesn't even include
nanocoap.h
, which is the main dependency.Thus, this PR proposes to declare such dependencies in Makefile.dep