Skip to content

[WIP] Move to core+alloc #189

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
Draft
daviddrysdale wants to merge 18 commits into
base: main
Choose a base branch
from
Draft

[WIP] Move to core+alloc #189

daviddrysdale wants to merge 18 commits into from

Conversation

@daviddrysdale
Copy link
Collaborator

@daviddrysdale daviddrysdale commented Jul 19, 2021

Fixes #7

@daviddrysdale daviddrysdale force-pushed the no-std branch 7 times, most recently from 7db1bf3 to 2eb5c31 Compare July 19, 2021 12:41
@daviddrysdale
Copy link
Collaborator Author

daviddrysdale commented Jul 19, 2021 •
edited
Loading

This currently still pulls in std because:

@daviddrysdale
tink-proto: move to core+alloc
17bb6be 
The "json" feature requires the base64 and serde dependencies to have
their "alloc" feature enabled.
@daviddrysdale
tink-core: move to core+alloc with std feature
6613df9 
Requires use of `rand::OsRng` (and with it the `getrandom` feature of
the `rand` crate) rather than `rand::thread_rng()`.
@daviddrysdale
tink-core: expect Display not Error for sub-errs
c140938
@daviddrysdale
tink-prf: move to core+alloc
a6b29d3
@daviddrysdale
tink-mac: move to core+alloc
6cef433
@daviddrysdale
tink-aead: move to core+alloc
024b71c
@daviddrysdale
tink-daead: move to core+alloc
c3571fd
@daviddrysdale
tink-streaming-aead: update deps for std use
2287fc7
@daviddrysdale
tink-signature: move to core+alloc
528da2e 
This means that there are no details in errors, so remove the test for
specific error text.
@daviddrysdale
tink-awskms: update deps
a659054
@daviddrysdale
tink-gcpkms: update deps
883a7a9
@daviddrysdale
rinkey: update deps
7424ff7
@daviddrysdale
tink-tests: update deps for std usage
044dc26 
Move to use the same features of dependencies as the crates under test.
This requires a shift from rand::thread_rng() to rand::rngs::OsRng.

This crate still uses `std` though.
@daviddrysdale
tink-testing: update deps
80c826a
@daviddrysdale
examples: update deps
e05b038
@daviddrysdale
ci: run clippy over feature combinations
dd1518d
@daviddrysdale
no-std: add binary crate to check no_std compliance
9010159
@daviddrysdale
temp: use forked ed25519-dalek, bump rand
66e302b 
This commit cannot be included in a release, because it makes the codebase
rely on an unreleased crate.
@daviddrysdale daviddrysdale marked this pull request as draft November 14, 2021 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support no_std in (most) crates

1 participant

@daviddrysdale