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

Compilation error #5432

Closed
carstein opened this issue Jan 6, 2023 · 6 comments · Fixed by #5570
Closed

Compilation error #5432

carstein opened this issue Jan 6, 2023 · 6 comments · Fixed by #5570
Labels
C-bug Category: This is a bug

Comments

@carstein
Copy link
Contributor

carstein commented Jan 6, 2023

Summary

Currently the project does not compile because of the 'ignore v0.4.19' package - this package requires 'scoped_threads' - an unstable feature that only stabilized in Rust 1.63. Unfortunatelly the rust-toolchain.toml overrides the current version down to 1.61. That causes the compilation to fail.

This can be fixed by bumping version up to at least 1.63.0

Reproduction Steps

$ git clone https://github.com/helix-editor/helix helix
$ cd helix
$ cargo install --path helix-term

Helix log

No response

Platform

Linux

Terminal Emulator

gnome-terminal

Helix Version

22.12-95-gf2fff9b9

@carstein carstein added the C-bug Category: This is a bug label Jan 6, 2023
@the-mikedavis
Copy link
Member

ignore is currently locked at 0.4.18:

helix/Cargo.lock

Lines 1353 to 1355 in f2fff9b

[[package]]
name = "ignore"
version = "0.4.18"

@carstein
Copy link
Contributor Author

carstein commented Jan 6, 2023

And yet ...
Compiling grep-searcher v0.1.11
Compiling ignore v0.4.19
Compiling grep-regex v0.1.11

@carstein
Copy link
Contributor Author

carstein commented Jan 6, 2023

After reading some more - cargo install by default ignores Cargo.lock - you can force the behaviour by running it with --locked

@the-mikedavis
Copy link
Member

Oh interesting, I'm surprised that cargo behaves that way. The docs for installation from source should include the --locked flag to prevent this snag. Would you like to submit a PR?

@carstein
Copy link
Contributor Author

carstein commented Jan 7, 2023

Done: #5438

@pascalkuthe
Copy link
Member

The next firefox release wil happen on 2023/1/17 so we can raise the MSRV to 1.63 soon to fix this problem (although encouraging the use of --locked is still good regardless)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants