8000 Allow building with the Arduino core, but without a .pde or .ino. by peplin · Pull Request #105 · sudar/Arduino-Makefile · GitHub
[go: up one dir, main page]

Skip to content

Allow building with the Arduino core, but without a .pde or .ino. #105

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

Closed
wants to merge 1 commit into from
Closed

Allow building with the Arduino core, but without a .pde or .ino. #105

wants to merge 1 commit into from

Conversation

peplin
Copy link
Contributor
@peplin peplin commented Jul 9, 2013

There's no particular reason those two things need to be associated -
you should be able to compile a project that has only .c/.cpp files and
still use the core libraries.

There's no particular reason those two things need to be associated -
you should be able to compile a project that has only .c/.cpp files and
still use the core libraries.
@sudar
Copy link
Owner
sudar commented Jul 27, 2013

@peplin

Are you trying to solve the usecase where you want to compile projects that have only *.c or *.cpp code without any *.ino or *.pde files, but still want to include arduino core?

The NO_CORE option that I talked about does the first part (compile *.c and *.cpp), but doesn't include arduino core.

If you confirm your usecase, then I will try to find a better solution for it.

This reminds me of an open issue #39 that we have. If there are more than one source file (*.ino or *.pde or *.c{pp}) the symbols defined in one file is not available in the other one. Since you are using more than one file for your cantranslator project, how did you solve it there?

@peplin
Copy link
Contributor Author
peplin commented Jul 29, 2013

That's correct, there's no .ino or *.pde but we *do want to include the Arduino core.

We get around #39 by using typical header files and not relying on any of the behind-the-scenes Arduino magic, particularly because the same code base is compiled for ARM with a different toolchain as well.

@sudar
Copy link
Owner
sudar commented Jul 29, 2013

@peplin,

Thanks for confirming it.

Let me see if I can combine your code and NO_CORE check. If I am not able to do it, then I will accept this pull request.

@sudar sudar closed this in d719847 Aug 9, 2013
@sudar
Copy link
Owner
sudar commented Aug 9, 2013

@peplin

Kindly check commits c92d672 and d719847 and let me know whether this will fix the problem that you were facing.

@peplin
Copy link
Contributor Author
peplin commented Aug 9, 2013

Good to go, it works fine - thanks.

@peplin peplin deleted the no-suffix-but-keep-core branch August 9, 2013 21:09
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
0