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.

add discovered arduino library dependencies as link targets to genera… #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RahmanQureshi
Copy link

…ted arduino libraries.

I was building a project involving compiling RadioHead which uses the arduino SPI library. I compiled it as a static library first. It compiled fine but linking failed with unresolved references. Simply swapping the ordering of the SPI static library file allowed it to compile.

Looking into the code, the dependency on uno_SPI library wasn't being set without this change.

Maybe I'm using it wrong though,

@MrPointer
Copy link
Member

Hi @RahmanQureshi, I appreciate your work, thanks for the effort!
However, this repo and version of Arduino-CMake is no longer support by the maintainers (including myself). Please see #77 (comment) which refers to a newer version which will hopefully resolve your issue.
Thus, I simply can't accept the current PR, but I'm leaving it open in case it does help someone else.
Thanks again! 😃

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