Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on May 4, 2019. It is now read-only.

fixed arduino library includes #31

Merged
merged 1 commit into from
Dec 8, 2017
Merged

fixed arduino library includes #31

merged 1 commit into from
Dec 8, 2017

Conversation

fraillt
Copy link

@fraillt fraillt commented Dec 8, 2017

  • find_arduino_library was fixed, it falsely included local directory as a library, even though it was not compatible with Arduino library structure. Now it works as is written in the comment.

A Arduino library is a folder that has the same name as the include header.
For example, if we have a include "#include <LibraryName.h>" then the following
directory structure is considered an Arduino library:

LibraryName/
|- LibraryName.h
`- LibraryName.c

  • make_arduino_library was fixed, it didn't return include path for libraries that had a cmake target already created.

@fraillt fraillt requested a review from MrPointer December 8, 2017 06:21
@MrPointer MrPointer merged commit 1b486d5 into arduino-cmake:release/2.0.0 Dec 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants