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.

Sign up for GitHub

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

Jump to bottom

implementations||adopters: point to C and Rust libraries #896

Merged
merged 1 commit into from
Feb 24, 2022
Merged

implementations||adopters: point to C and Rust libraries #896

merged 1 commit into from
Feb 24, 2022

Conversation

vbatts
Copy link
Member

@vbatts vbatts commented Feb 16, 2022

having seen #895, it's worth ensuring these other languages are listed
implementations

Signed-off-by: Vincent Batts vbatts@hashbangbash.com

@vbatts
implementations: adding the C and Rust libraries
02c5c05 
having seen opencontainers#895, it's worth ensuring these other languages are listed
implementations

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts vbatts mentioned this pull request Feb 16, 2022
Closed
vrothberg
vrothberg reviewed Feb 16, 2022
View reviewed changes
Copy link
Contributor

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cardil
Copy link

cardil commented Feb 16, 2022

I've opened the #895, having Rust in mind. I end up using krustlet/oci-distribution instead, of containers/oci-spec-rs. The containers/oci-spec-rs is really basic - you could only parse manifest file with it, which isn't much.

The krustlet/oci-distribution has a far greater scope. It's still under heavy development, though. Here's a code I just wrote that fetches a remote image and saves it in the user's home:

https://github.com/cardil/ocilot/blob/d5ac007e5804bd9c2f5b353ccb29300d337446a0/oci/src/registry.rs#L21-L38

There are is also:

@vbatts vbatts changed the title implementations: adding the C and Rust libraries implementations||adopters: point to C and Rust libraries Feb 24, 2022
@vbatts vbatts merged commit 1f308fd into opencontainers:main Feb 24, 2022
@vbatts vbatts deleted the implementations branch February 24, 2022 18:51
@vbatts
Copy link
Member Author

vbatts commented Feb 24, 2022

this merged fast during the call, but I've opened #897 to track the krustlet library you mentioned, @cardil

@sudo-bmitch sudo-bmitch mentioned this pull request Sep 16, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vrothberg vrothberg vrothberg left review comments

@jonjohnsonjr jonjohnsonjr jonjohnsonjr approved these changes

@stevvooe stevvooe stevvooe approved these changes

@brendandburns brendandburns Awaiting requested review from brendandburns

@jstarks jstarks Awaiting requested review from jstarks

@jonboulle jonboulle Awaiting requested review from jonboulle

@cyphar cyphar Awaiting requested review from cyphar cyphar is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@vbatts @cardil @stevvooe @vrothberg @jonjohnsonjr