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

rust: Freeze rand crate to v0.6.1 #1778

Closed
wants to merge 2 commits into from
Closed

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Mar 7, 2019

The recent dep bump caused rand to jump to v0.6.5, which pulled in
fuchsia-cprng. Sadly, the cargo version we're frozen on chokes on it
because its manifest makes use of the edition keyword, which wasn't
stabilized yet.

Let's just freeze the problematic crate for now. We should be able to
bump our Rust toolchain soon-ish.

(Note this commit also does some other minor crate bumps as a result of
regenerating the lockfile to prune away fuchsia-cprng.)

Closes: #1777

jlebon added 2 commits March 7, 2019 15:40
The recent dep bump caused rand to jump to v0.6.5, which pulled in
`fuchsia-cprng`. Sadly, the cargo version we're frozen on chokes on it
because its manifest makes use of the `edition` keyword, which wasn't
stabilized yet.

Let's just freeze the problematic crate for now. We should be able to
bump our Rust toolchain soon-ish.

(Note this commit also does some other minor crate bumps as a result of
regenerating the lockfile to prune away `fuchsia-cprng`.)

Closes: coreos#1777
Otherwise, we're not testing everything. E.g. the cbindgen bits.
@cgwalters
Copy link
Member

@rh-atomic-bot r+ 0ec4cc0

@rh-atomic-bot
Copy link

⌛ Testing commit 0ec4cc0 with merge b39439f...

rh-atomic-bot pushed a commit that referenced this pull request Mar 7, 2019
The recent dep bump caused rand to jump to v0.6.5, which pulled in
`fuchsia-cprng`. Sadly, the cargo version we're frozen on chokes on it
because its manifest makes use of the `edition` keyword, which wasn't
stabilized yet.

Let's just freeze the problematic crate for now. We should be able to
bump our Rust toolchain soon-ish.

(Note this commit also does some other minor crate bumps as a result of
regenerating the lockfile to prune away `fuchsia-cprng`.)

Closes: #1777

Closes: #1778
Approved by: cgwalters
rh-atomic-bot pushed a commit that referenced this pull request Mar 7, 2019
Otherwise, we're not testing everything. E.g. the cbindgen bits.

Closes: #1778
Approved by: cgwalters
@jlebon
Copy link
Member Author

jlebon commented Mar 7, 2019

@rh-atomic-bot retry

rh-atomic-bot pushed a commit that referenced this pull request Mar 7, 2019
Otherwise, we're not testing everything. E.g. the cbindgen bits.

Closes: #1778
Approved by: cgwalters
@rh-atomic-bot
Copy link

⌛ Testing commit 0ec4cc0 with merge 6b928e8...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: cgwalters
Pushing 6b928e8 to master...

@jlebon jlebon deleted the pr/fix-rand branch April 23, 2023 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants