Skip to content

Conversation

pfeerick
Copy link

@pfeerick pfeerick commented Jan 5, 2019

Minimum required library.json to allow easy install/management with PlatformIO library manager

Minimum required library.json to allow easy install/management with PlatformIO library manager
@TheNitek
Copy link

Any chance to merge this? Would be awesome!

@pfeerick
Copy link
Author

pfeerick commented Aug 19, 2019

In the meantime, it can be easily added via the git repo reference... i.e. lib_deps = markruys/arduino-Max72xxPanel will work just fine, as long as you have git installed. It would just be nice for it to come up in the Library manager when people search for libraries. To be honest, I'd prefer to see #17 accepted over this PR as I've since come to realise the PlatformIO Library Registry can work with that file as well, making it so the author doesn't need to maintain two or more library manifiest files.

vortigont added a commit to vortigont/arduino-Max72xxPanel that referenced this pull request Nov 1, 2020
@rob040
Copy link

rob040 commented Jun 29, 2025

In the meantime, it can be easily added via the git repo reference... i.e. lib_deps = markruys/arduino-Max72xxPanel will work just fine, as long as you have git installed. It would just be nice for it to come up in the Library manager when people search for libraries.

Currently (6 years later) it does not (anymore?).

Adding lib_deps = markruys/arduino-Max72xxPanel to platformio.ini will cause the following error:

UnknownPackageError: Could not find the package with 'markruys/arduino-Max72xxPanel' requirements for your system 'windows_amd64'

Specifying as
lib_deps = https://github.com/markruys/arduino-Max72xxPanel
Does solve that, but the fact that this library is missing version information and was never released and is unmaintained for 12 years is not a good recommendation.

@markruys please implement #16 and #17

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.

3 participants