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

Drop cap-std from our public APIs #3027

Merged
merged 3 commits into from
Aug 31, 2023

Conversation

cgwalters
Copy link
Member

Since it bumped semver (when I didn't expect it to; xref bytecodealliance/cap-std@963eebf#r121651362

It's not load-bearing enough here to matter versus just passing an untyped file descriptor.

This mainly means that it will be the glib ecosystem which forces transitive semver bumps for us, not both.

@cgwalters cgwalters added area/rust-bindings Relates to the Rust bindings for the C library needs-work/ci-failure Legitimate CI failure labels Aug 31, 2023
@cgwalters cgwalters force-pushed the drop-cap-std-public branch 2 times, most recently from 29df5bf to 135b40c Compare August 31, 2023 18:53
@cgwalters cgwalters removed the needs-work/ci-failure Legitimate CI failure label Aug 31, 2023
Since it bumped semver (when I didn't expect it to; xref
bytecodealliance/cap-std@963eebf#r121651362

It's not load-bearing enough here to matter versus just passing
an untyped file descriptor.

This mainly means that it will be the `glib` ecosystem which
forces transitive semver bumps for us, not both.
To pick up the new `AsFd` etc.
And drop another hardcoded MSRV.
Copy link
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

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

lgtm

@jmarrero jmarrero merged commit 5fe050f into ostreedev:main Aug 31, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rust-bindings Relates to the Rust bindings for the C library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants