-
Notifications
You must be signed in to change notification settings - Fork 449
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
Arduino.mk insists on building way too many unnecessary libraries #29
Comments
Hmm, I see. It's just grabbing everything in ARDUINO_CORE_PATH, and not making any decisions as to which core files make any sense for the target device. |
It's doing the same thing to me, building all of the core libraries. |
yup, i just noticed this too - Stream, WMath, Tone, HardwareSerial, USBCore etc.even for Blink. they're not getting put into the hex though (800k of libs, 4k hex!) so its just a waste of compile time rather than arduino memory. |
I just enabled verbose logs in Arduino IDE to see how Arduino handles it. It seems even Arduino IDE tries to compile everything, but link only the necessary stuff as @sej7278 found out. I am thinking of marking it as Here is the (rather long) output from Arduino IDE
|
I noticed that, forgot to update the issue, sorry. It is irritating but not a huge problem; as the linker filters all that stuff out. Your call if you want to keep it open or not. |
I am marking this issue as The fix for this issue will be "nice to have", and that's why I am not marking it as Would be happy to merge, if someone can implement it. But I am not sure, if it can be implemented. |
yeah thats fine as the IDE does it that way too, also upon a 2nd pass of make it doesn't rebuild everything |
I have this simple makefile
And make seems to insist on building all kinds of insane stuff, like HID and USB, and IPAddress???!, when I did not tell it to do that.
Why does it think that a diecimilla is a Leonardo? It isn't. I've been looking around in the makefile to try to find out where this mistake is coming from, and can't seem to find it anywhere.
The text was updated successfully, but these errors were encountered: