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

README.md: explain how to edit libraries #21

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

Conversation

glasser
Copy link

@glasser glasser commented Oct 17, 2017

No description provided.

@obra
Copy link
Member

obra commented Oct 17, 2017

Having advice about this here is reasonable, but I don't think that this is the right advice. (Well, I desperately hope that this is not the right advice.)

I'll test what I believe to be the correct solution today and propose an update.

@Jennigma
Copy link
Contributor

I mentioned in the thread on the community board that I encountered a multiple copies of a library compile time warning through the IDE, and it appeared to be respecting the order in $PATH when it chose which one to include. Should be easy enough to talk folks through changing that variable, though I have no idea how pc's handle such things.

@algernon
Copy link
Contributor

it appeared to be respecting the order in $PATH when it chose which one to include.

It's sadly much more complicated than that. I went to figure this out once, and I seem to recall documenting my findings in an issue somewhere... will try to dig that up, perhaps it will prove helpful for others.

@algernon
Copy link
Contributor

Had a look, and this was on IRC, before logging was set up. The gist of things is that built-in libraries can be overridden by others, the order - if my IRC notes are to be trusted - depends on the order of -libraries <path> passed to arduino-builder.

Built-in libraries are those that come with Arduino, and libraries under the hardware hierarchy.

Perhaps it isn't that complicated after all...

@obra
Copy link
Member

obra commented Oct 17, 2017 via email

@obra
Copy link
Member

obra commented Aug 11, 2018

@algernon - do we have a better answer for this these days?

@algernon
Copy link
Contributor

I do not think we have a better answer :(

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.

4 participants