Skip to content
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

Restructuring project dirs and files #24

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

a-andreyev
Copy link

Hello!
How about to separate libraries and microcoap-example folders to simplify building on both platforms?
I implemented the pull request (used actual version), changed coap to microcoap at some headers.
Also I separated enpoints header to simplify the new coap links implementing and added the info to readme file.
Changed the arduino pin from 6 to 13 (default embedded led at some arduino boards) and moved the definition to /microcoap-example/connections.h
Tested with arduino mega and with the linux-based OS.

@tobyjaffey
Copy link
Contributor

I like the structure, but it doesn't compile.

cc -Wall -DDEBUG -Ilibraries/microcoap microcoap-example -c main-posix.c -o main-posix.o
clang: warning: microcoap-example: 'linker' input unused
main-posix.c:7:10: fatal error: 'coap.h' file not found
#include "coap.h"
^
1 error generated.
make: *** [main-posix.o] Error 1

@a-andreyev
Copy link
Author

Thank you for your fast response!
Sorry for the mistakes with the previous pull request.
Does it compiles now?

For me current version is ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants