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

Unique name value in library.properties #23

Merged
merged 1 commit into from
Jan 31, 2016
Merged

Unique name value in library.properties #23

merged 1 commit into from
Jan 31, 2016

Conversation

per1234
Copy link

@per1234 per1234 commented Oct 16, 2015

This pull request fixes a problem with some of the bundled libraries and the Arduino IDE Library Manager.

SD, Servo, and Firmata libraries were appearing as Updatable in Library Manager, and triggering updatable library notifications in Arduino IDE 1.6.6+, whenever a Mighty 1284P board was selected because they have the same name value(because if no library.properties file is present then the folder name is used). I used version 1.0.0 in library.properties for SD, Servo, and SDfat libraries because I was unable to determine which version they were based on.

This issue exacerbates #24 because it can cause the user to unnecessarily install the unpatched libraries to their {sketchbook}/libraries folder. This is especially likely with Arduino IDE versions 1.6.6+ because of the updatable library notifications.

Libraries having the same name value as the stock Arduino library in library.properties causes the libraries to always appear as type Updatable in Library Manager.
@per1234 per1234 changed the title Change libraries to Arduino IDE 1.5 format Unique name value in library.properties Jan 30, 2016
@per1234
Copy link
Author

per1234 commented Jan 30, 2016

I have reworked this PR to be minimally invasive while still fixing the issue of always updatable libraries.

pico-- added a commit that referenced this pull request Jan 31, 2016
Unique name value in library.properties
@pico-- pico-- merged commit 1cca804 into JChristensen:v1.6.3 Jan 31, 2016
@pico--
Copy link
Collaborator

pico-- commented Jan 31, 2016

Merged. Thanks, Per.

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.

2 participants