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

Add to ESP Component Registry #227

Closed
manforowicz opened this issue Aug 29, 2024 · 8 comments · Fixed by #229
Closed

Add to ESP Component Registry #227

manforowicz opened this issue Aug 29, 2024 · 8 comments · Fixed by #229

Comments

@manforowicz
Copy link
Contributor

What do you think about adding libcanard to the ESP Component Registry?
Since the ESP32 has great CAN bus support, having libcanard packaged in an ESP component would make it more accessible to developers.

Doing this would probably entail:

  1. Requesting an OpenCyphal namespace through the ESP Registry permissions page. Giving key contributors access to this namespace.

  2. Packaging up libcanard by following the Packaging ESP-IDF Components guide. This requires adding some metadata and build files.

  3. Uploading the package to the ESP registry each time there's an update.

@pavel-kirienko
Copy link
Member

pavel-kirienko commented Aug 29, 2024 via email

@kumekay
Copy link

kumekay commented Sep 2, 2024

Hello @manforowicz, I'm Sergei from Espressif, thank you so much for sending your request for namespace opencyphal in the ESP Component Registry. Since you are not an active contributor to the project, before approving your request, I decided to discuss the topic here.

@pavel-kirienko if you want a full control over components in the ESP Component Registry, could you please sign up here: https://components.espressif.com/users/login? Then you will be able to give permissions to other developers.

BTW, for ease of uploading I would highly recommend using our GitHub Action https://github.com/espressif/upload-components-ci-action

@XDanielPaul FYI

@pavel-kirienko
Copy link
Member

Thank you @kumekay for the advice. I have attempted to request the opencyphal namespace but it's already taken, presumably by @manforowicz. Could you please transfer it to me as the owner and then I will share access with the other team members? Thank you!

image

@pavel-kirienko
Copy link
Member

I also requested cyphal for security reasons.

@XDanielPaul
Copy link

Hello @pavel-kirienko, I'm Daniel from Espressif.
I have approved your namespace request for the namespace cyphal and would further want to ask you to request the namespace opencyphal once again in the ESP Component Registry, it should be available now :)

@pavel-kirienko
Copy link
Member

Thank you @XDanielPaul, this has been done.

@thirtytwobits FYI -- you don't seem to be registered in the ESP registry yet

@pavel-kirienko
Copy link
Member

@manforowicz a pull request amending the GitHub actions pipelines as necessary would be accepted.

@XDanielPaul
Copy link

XDanielPaul commented Sep 3, 2024

Thank you so much @pavel-kirienko, I have approved your namespace request for the opencyphal as well. You should be able to add other users to the namespaces now

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 a pull request may close this issue.

4 participants