Skip to content

Conversation

@dburgener
Copy link

Home released version 5.11 bumping the msrv to rust 1.81. They justify such an aggressive msrv policy by stating that users should not be using the home crate at all, but instead should rely on the standard library home_dir function. That function has a bug that is not fixed until 1.85, which prevents it from being an option for lower rust versions.

To mitigate, just pin to an older home version for now.

Home released version 5.11 bumping the msrv to rust 1.81.  They justify
such an aggressive msrv policy by stating that users should not be using
the home crate at all, but instead should rely on the standard library
home_dir function.  That function has a bug that is not fixed until
1.85, which prevents it from being an option for lower rust versions.

To mitigate, just pin to an older home version for now.
dburgener added a commit to dburgener/lalrpop that referenced this pull request Dec 17, 2024
This works around the home MSRV bump until a long term solution is
available.

See discussion at:

lalrpop#1015
Stebalien/term#123
rust-lang/cargo#13270
dburgener added a commit to dburgener/lalrpop that referenced this pull request Dec 17, 2024
This works around the home MSRV bump until a long term solution is
available.

See discussion at:

lalrpop#1015
Stebalien/term#123
rust-lang/cargo#13270
@Stebalien
Copy link
Owner

You can downgrade it in your own crate. If I cap this here, nobody else will be able to upgrade it if they want the latest version.

@Stebalien Stebalien closed this Dec 17, 2024
@dburgener dburgener deleted the home-dep branch December 17, 2024 16:12
@dburgener
Copy link
Author

Thanks for your consideration. Will you be bumping the msrv for this crate to 1.81? It no longer builds on versions below 1.81 due to the home dependency.

@Stebalien
Copy link
Owner

You can build this crate with 1.63, you just need to constrain the home version in your Cargo.toml. I know it kind of sucks, but this is why the cargo team is working on making cargo MSRV aware.

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