[WIP] Move to core+alloc #189

daviddrysdale · 2021-07-19T07:15:36Z

Fixes #7

daviddrysdale · 2021-07-19T14:54:51Z

This currently still pulls in std because:

tink-signature depends on ed25519-dalek ^1.0.1
- ed25519-dalek 1.0.1 depends on rand_core ^0.5, we (separately) turn on the getrandom feature
  - rand_core 0.5.1 depends on getrandom ^0.1
    - getrandom 0.1.16 pulls in std on most common platforms, even when the std feature is absent

The "json" feature requires the base64 and serde dependencies to have their "alloc" feature enabled.

Requires use of `rand::OsRng` (and with it the `getrandom` feature of the `rand` crate) rather than `rand::thread_rng()`.

This means that there are no details in errors, so remove the test for specific error text.

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.

This commit cannot be included in a release, because it makes the codebase rely on an unreleased crate.

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

daviddrysdale force-pushed the no-std branch from 5699303 to 8aab20e Compare July 21, 2021 09:59

daviddrysdale added 18 commits September 20, 2021 08:40

tink-proto: move to core+alloc

17bb6be

The "json" feature requires the base64 and serde dependencies to have their "alloc" feature enabled.

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()`.

tink-core: expect Display not Error for sub-errs

c140938

tink-prf: move to core+alloc

a6b29d3

tink-mac: move to core+alloc

6cef433

tink-aead: move to core+alloc

024b71c

tink-daead: move to core+alloc

c3571fd

tink-streaming-aead: update deps for std use

2287fc7

tink-signature: move to core+alloc

528da2e

This means that there are no details in errors, so remove the test for specific error text.

tink-awskms: update deps

a659054

tink-gcpkms: update deps

883a7a9

rinkey: update deps

7424ff7

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.

tink-testing: update deps

80c826a

examples: update deps

e05b038

ci: run clippy over feature combinations

dd1518d

no-std: add binary crate to check no_std compliance

9010159

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 force-pushed the no-std branch from 8aab20e to 66e302b Compare September 20, 2021 07:53

daviddrysdale marked this pull request as draft November 14, 2021 15:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[WIP] Move to core+alloc #189

[WIP] Move to core+alloc #189

daviddrysdale commented Jul 19, 2021

daviddrysdale commented Jul 19, 2021 •

edited

Loading

[WIP] Move to core+alloc #189

Are you sure you want to change the base?

[WIP] Move to core+alloc #189

Conversation

daviddrysdale commented Jul 19, 2021

daviddrysdale commented Jul 19, 2021 • edited Loading

daviddrysdale commented Jul 19, 2021 •

edited

Loading