-
Notifications
You must be signed in to change notification settings - Fork 227
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
docs(README): wrong crate name in cargo install
instructions
#964
docs(README): wrong crate name in cargo install
instructions
#964
Conversation
@@ -95,7 +95,7 @@ sudo zypper install linutil | |||
Linutil can be installed via [Cargo](https://doc.rust-lang.org/cargo) with: | |||
|
|||
```bash | |||
cargo install linutil | |||
cargo install linutil_tui |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably add --locked
to it as well since the version currently on crates cannot compile because one of the dependencies broke semver. Using --locked
will have the install only use the dependent versions that are in the Cargo.lock file rather than the latest semver matching ones.
(The broken dependency really needs to fix that, cargo-semver-checks
exists for a reason after all, lol)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing that out. I really have no idea what's going on with versioning here. I'll try to fix all this today but @ChrisTitusTech IMO you really should check if crates 100% work before releasing.
I'm not sure if this will work though, as Cargo.lock
is not included in the packages.
Line 10 in ab7a670
include = ["src/*.rs", "Cargo.toml", "cool_tips.txt", "../man/linutil.1"] |
Line 8 in ab7a670
include = ["src/*.rs", "Cargo.toml", "tabs/**"] |
@ChrisTitusTech Why is the version in Line 3 in 00914ab
|
Release workflow should be modified to update versions in Cargo.toml |
#965 will resolve this issue. |
Resolves #963