Skip to content

Conversation

@musicinmybrain
Copy link
Contributor

Pin home to <0.5.11 to avoid MSRV problems. As described in https://github.com/lunacookies/etcetera/releases/tag/v0.10.0. This keeps the MSRV at 1.70.0 (less than fd’s 1.77.2), rather than increasing it to 1.81.0 for etcetera 0.10.

@tavianator
Copy link
Collaborator

Thanks! I PR'd the clippy fix as #1712, we can rebase this once that's merged

Cargo.toml Outdated
# Temporarily pin home to keep the MSRV of etcetera from increasing to 1.81.0.
# This direct dependency can be removed after our MSRV is at least 1.81.0.
# https://github.com/lunacookies/etcetera/releases/tag/v0.10.0
home = "<0.5.11"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't really work as desired. If I run cargo update I get a duplicated home dependency, and the MSRV is still not respected:

├── etcetera v0.10.0
│   ├── cfg-if v1.0.0
│   └── home v0.5.11
├── home v0.4.2

- Pin home to <0.5.11 to avoid MSRV problems
@tavianator tavianator merged commit 0d680b4 into sharkdp:master Apr 29, 2025
18 checks passed
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.

2 participants