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

IDF Component registry #987

Closed
nicklasb opened this issue Feb 26, 2024 · 2 comments · Fixed by #990
Closed

IDF Component registry #987

nicklasb opened this issue Feb 26, 2024 · 2 comments · Fixed by #990
Labels
enhancement New feature or request

Comments

@nicklasb
Copy link
Contributor

nicklasb commented Feb 26, 2024

Hi,
Saw that you added an ESP-IDF build step, just saying that it feels like you'd have very little more to do to add Radiolib to the Espressif IDF component library to make it easy for my library to depend on it there.

I'd rather not have to make a dumb fork of my own (with the exact same content) and confuse everybody.
Also, the IDF registry doesn't allow git repository references (for a pretty good reason), so my library I cannot depend on it that way either.

Anyway, it is not that hard, and would help the glorious Radiolib and its charming maintainer bedazzle and rule them all even more. :-)

@nicklasb
Copy link
Contributor Author

Would you like a PR for that?

@jgromes
Copy link
Owner

jgromes commented Feb 27, 2024

I don't use ESP-IDF that much, so I'm not that familiar with the whole ecosystem - could you elaborate on what would be needed to add RadioLib to Espressif IDF component library? I imagine it's something like the Platformio package index ...?

By chance, I did recently start one proejct using ESP-IDF and RadioLib, so I did make it work there (pretty much just using the NonArduino/ESP-IDF example as the basis), but like I said I'm not really that familiar with ESP-IDF, so I have no idea if the way it's handled currently is correct.

Would you like a PR for that?

If it makes the library easier to use in ESP-IDF, then that would be great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants