Skip to content

Update parking lot to 0.11#5

Merged
Amanieu merged 2 commits into
Amanieu:masterfrom
nico-abram:update_parking_lot
Oct 27, 2020
Merged

Update parking lot to 0.11#5
Amanieu merged 2 commits into
Amanieu:masterfrom
nico-abram:update_parking_lot

Conversation

@nico-abram

Copy link
Copy Markdown
Contributor

@nico-abram

Copy link
Copy Markdown
Contributor Author

CI seems to fail for the same reason as #1 (comment)

@Amanieu

Amanieu commented Oct 25, 2020

Copy link
Copy Markdown
Owner

Just bump the minimum Rust version in CI to fix the issue.

nico-abram added a commit to nico-abram/seqlock that referenced this pull request Oct 25, 2020
nico-abram added a commit to nico-abram/seqlock that referenced this pull request Oct 25, 2020
@nico-abram

Copy link
Copy Markdown
Contributor Author

Done! I skimmed the readme and didn't see any explicit mention of a MSRV to update

I used 1.31 which I think is the first rust 2018 edition which I think is required for smallvec since it's using use crate:: (2018 edition mentions use changes here and this was the travis run that broke despite having the AtomicUsize method that was previously crashing)

@nico-abram

nico-abram commented Oct 25, 2020

Copy link
Copy Markdown
Contributor Author

Oh, it seems smallvec is now breaking on const generics https://travis-ci.org/github/Amanieu/seqlock/jobs/738652796#L240

nico-abram added a commit to nico-abram/seqlock that referenced this pull request Oct 25, 2020
@nico-abram

Copy link
Copy Markdown
Contributor Author

I bumped it to 1.36 which seems to be what parking_lot uses in its CI https://github.com/Amanieu/parking_lot/blob/master/.github/workflows/rust.yml#L19

@Amanieu Amanieu merged commit 61fa8cf into Amanieu:master Oct 27, 2020
@Amanieu

Amanieu commented Oct 27, 2020

Copy link
Copy Markdown
Owner

Thanks!

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.

Panic when running benchmark

2 participants