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

elliptic-curve: add ff and group crate dependencies #287

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Sep 4, 2020

Adds the ff and group crates as dependencies, with the goal of replacing some of the existing traits and trait relationships in elliptic-curve with ones from ff/group.

This PR also removes the rand and weierstrass features from the elliptic-curve crate, making rand_core a hard requirement, since it seems at least rand_core will probably be a hard requirement for group, and requiring an RNG as a hard dependency makes sense.

Adds the `ff` and `group` crates as dependencies, with the goal of
replacing some of the existing traits and trait relationships in
`elliptic-curve` with ones from `ff`/`group`.

This PR also removes the `rand` and `weierstrass` features from the
`elliptic-curve` crate, making `rand_core` a hard requirement, since it
seems at least `rand_core` will probably be a hard requirement for
`group`, and requiring an RNG as a hard dependency makes sense.
@tarcieri tarcieri force-pushed the elliptic-curve/ff-and-group-crates branch from 7c78793 to 10bd8fd Compare September 4, 2020 14:22
@tarcieri tarcieri merged commit 80abd0a into master Sep 4, 2020
@tarcieri tarcieri deleted the elliptic-curve/ff-and-group-crates branch September 4, 2020 14:37
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Sep 4, 2020
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Sep 4, 2020
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 4, 2020
Updates to use the `group` crate. See: RustCrypto/traits#287.

This crate has a hard `rand_core` dependency so this commit gets rid of
the `rand` features across the board and makes them mandatory.

(Even if we don't end up shipping the `group` crate this release, that's
probably for the best to keep the number of features down)
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 4, 2020
Updates to use the `group` crate. See: RustCrypto/traits#287.

This crate has a hard `rand_core` dependency so this commit gets rid of
the `rand` features across the board and makes them mandatory.

(Even if we don't end up shipping the `group` crate this release, that's
probably for the best to keep the number of features down)
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 4, 2020
Updates to use the `group` crate. See: RustCrypto/traits#287.

This crate has a hard `rand_core` dependency so this commit gets rid of
the `rand` features across the board and makes them mandatory.

(Even if we don't end up shipping the `group` crate this release, that's
probably for the best to keep the number of features down)

This commit additionally splits out `no_std` build testing into
`tests/*_no_std` Cargo projects. This is a workaround until the Cargo
resolver is fixed upstream:

rust-lang/cargo#7915
rust-lang/cargo#7916
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 4, 2020
Updates to use the `group` crate. See: RustCrypto/traits#287.

This crate has a hard `rand_core` dependency so this commit gets rid of
the `rand` features across the board and makes them mandatory.

(Even if we don't end up shipping the `group` crate this release, that's
probably for the best to keep the number of features down)

This commit additionally splits out `no_std` build testing into
`tests/*_no_std` Cargo projects. This is a workaround until the Cargo
resolver is fixed upstream:

rust-lang/cargo#7915
rust-lang/cargo#7916
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 4, 2020
Updates to use the `group` crate. See: RustCrypto/traits#287.

This crate has a hard `rand_core` dependency so this commit gets rid of
the `rand` features across the board and makes them mandatory.

(Even if we don't end up shipping the `group` crate this release, that's
probably for the best to keep the number of features down)

This commit additionally splits out `no_std` build testing into
`tests/*_no_std` Cargo projects. This is a workaround until the Cargo
resolver is fixed upstream:

rust-lang/cargo#7915
rust-lang/cargo#7916
dns2utf8 pushed a commit to dns2utf8/traits that referenced this pull request Jan 24, 2023
Bumps [base64ct](https://github.com/RustCrypto/formats) from 1.4.0 to 1.4.1.
- [Release notes](https://github.com/RustCrypto/formats/releases)
- [Commits](RustCrypto/formats@base64ct/v1.4.0...base64ct/v1.4.1)

---
updated-dependencies:
- dependency-name: base64ct
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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 this pull request may close these issues.

1 participant