search

arduino-libraries / SD

SD Library for Arduino
http://arduino.cc/
GNU General Public License v3.0
182 stars 155 forks source link

SD library always ask to update itself #46

Closed Dravon5X closed 6 years ago

Dravon5X commented 6 years ago

this fix https://github.com/arduino-libraries/SD/issues/31 https://forum.arduino.cc/index.php?topic=515050.0

actually produce a bug, that every one user of arudino IDE is receiving notification after IDE start that update is available for SD library, but there is nothing to update. SD library is also always under Updateable libraries. Please revert and find a different way.

per1234 commented 6 years ago

It actually just exposes a long-standing bug: https://github.com/arduino/Arduino/issues/4189.

Library Manager should be able to correctly handle a change in the architectures value so the solution is to fix the underlying bug, rather than to bandaid over it by reverting https://github.com/arduino-libraries/SD/commit/9543f55fdb08cce6d4ac5798d3b6b7347981a03b.

I'm inclined say this should be closed as a duplicate of https://github.com/arduino/Arduino/issues/4189 but maybe it will nudge some action on the long neglected https://github.com/arduino/Arduino/issues/4189.

amdkt7 commented 6 years ago

Is there a way to fix this locally? Every time I open a new window my screen will suddenly freeze when the notification pops up, and I lose my place. It's getting really annoying.

cmaglie commented 6 years ago

This is a bug that surely be fixed in the IDE (and I'm working on it on this branch), but in the meantime let's alleviate the pain by restoring the architecture=* in the library.properties.

amdkt7 commented 6 years ago

architecture= in the library.properties? Is that something that I can change locally? I found the file with the "architecture=" line, but your comment is unclear to me. Thanks for you help.

cmaglie commented 6 years ago

Just update the library to version 1.2.2 in the library manager.

amdkt7 commented 6 years ago

Okay, just did. Last time I checked a few days ago it was not there. Thanks.