Skip to content

Replace lazy_static with once_cell#2263

Merged
cmyr merged 1 commit intolinebender:masterfrom
jplatte:once-cell
Oct 17, 2022
Merged

Replace lazy_static with once_cell#2263
cmyr merged 1 commit intolinebender:masterfrom
jplatte:once-cell

Conversation

@jplatte
Copy link
Member

@jplatte jplatte commented Sep 21, 2022

lazy_static is increasingly getting replaced by once_cell and it's nice not to have multiple dependencies for the same thing.

Copy link

@notgull notgull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only real issue I have with this is the inclusion of version 1.15.0 of OnceCell, which includes a dramatic MSRV bump to rustc 1.56. I'm not sure what the MSRV policy is for Druid, but if it's lower than 1.56 this could introduce an unnecessary bump.

This can be avoided by using ^1.14 instead.

@jplatte
Copy link
Member Author

jplatte commented Sep 21, 2022

I haven't seen an MSRV specified anywhere. Happy to reduce the minimum version of once_cell if maintainers prefer.

@raphlinus
Copy link
Contributor

Druid is experimental, so I have no problem advancing the MSRV. I'm happy to see stuff move forward.

@jplatte
Copy link
Member Author

jplatte commented Sep 26, 2022

So can this be merged then? :)

@jplatte
Copy link
Member Author

jplatte commented Oct 14, 2022

Rebased, and lowered the minimum version for once_cell because it doesn't really hurt.

Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure let's try this, and we'll see if anything breaks 💁

@cmyr cmyr merged commit 6ac266c into linebender:master Oct 17, 2022
@jplatte jplatte deleted the once-cell branch October 17, 2022 14:36
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.

4 participants