Skip to content

Unique library.properties names #94

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

Closed
wants to merge 1 commit into from
Closed

Unique library.properties names #94

wants to merge 1 commit into from

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Dec 31, 2015

Change name value in library.properties to unique values for all libraries that are in the Library Manager index.

  • Prevents incorrect updatable library notifications(Arduino IDE 1.6.6 and up).
  • Fixes the problem of Arduino IDE built in libraries always showing as Type: updatable in Library Manager when the Arduino IDE built in library version number is higher than the Arduino 101 library version number.

Similar changes have already been made to Galileo and Edison corelibs: intel/corelibs-galileo#16, intel/corelibs-galileo#18, intel/corelibs-edison@09ec7d1

Change name value in library.properties to unique values for all
libraries that are in the Library Manager index.
- Prevents incorrect updatable library notifications(Arduino IDE 1.6.6
and up).
- Fixes the problem of Arduino IDE built in libraries always showing as
Type: updatable in Library Manager when the Arduino IDE built in library
version number is higher than the Arduino 101 library version number.
@kitsunami
Copy link

@SidLeung Please review.

@per1234
Copy link
Contributor Author

per1234 commented Feb 25, 2016

It looks like the plan is to add support for Arduino 101 to the original libraries and then I assume at some point the libraries will be removed from this repository.

Here's a summary of the current status of the libraries affected by this PR:

So if all these libraries are to be removed from this repository before the next release there is no need to merge this PR but if there is to be a release with any of the libraries included then this change should still be made first.

@bigdinotech
Copy link
Contributor

Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants