NetEEPROM: Add library.properties #24
Conversation
A completely different NetEEPROM library was added to Library Manager so a unique library.properties name value for the Ariadne NetEEPROM library is necessary to avoid spurious updatable library notifications.
|
I wonder if it might be better to re-write Ariadne's NetEEPROM to use the included NetEEPROM and just rename our NetEEPROM into something else, if another layer of abstraction is needed, that is. What is your opinion on this? |
|
Greg Tan's NetEEPROM isn't actually included with the Arduino IDE. It's just in the index of libraries available for installation via Library Manager. Since it has the same name as the Ariadne NetEEPROM, Library Manager is fooled into thinking that Greg Tan's NetEEPROM is installed and a newer version is available for updating. Updatable library popup notifications were added in Arduino IDE 1.6.6 and these just keep coming until you update. I haven't looked at Greg Tan's library thoroughly enough to see how feasible it would be to use it to set the Ariadne network settings but this would require the user to install Greg Tan's NetEEPROM library as well as installing Ariadne so that would make the process of getting started using Ariadne a little more complex. |
|
I see. Thanks for the information and the pull request, I am merging this right away then. |
NetEEPROM: Add library.properties
A completely different NetEEPROM library was added to Library Manager so a unique library.properties name value for the Ariadne NetEEPROM library is necessary to avoid NetEEPROM appearing as Type: Updatable in Arduino IDE Library Manager and receiving spurious updatable library notifications in Arduino IDE 1.6.6+. If the "update" is installed it will override the Ariadne version of NetEEPROM.
I'm happy to make any changes requested and squash to a single commit.