Skip to content

Change installation method#62

Merged
ritzdorf merged 2 commits intomainfrom
change_installation_method
May 16, 2025
Merged

Change installation method#62
ritzdorf merged 2 commits intomainfrom
change_installation_method

Conversation

@stiefn
Copy link
Contributor

@stiefn stiefn commented May 15, 2025

Change the installation instructions in the README

For some obscure reason, cargo build works, while cargo install throws an error on the same codebase.

@stiefn stiefn force-pushed the change_installation_method branch from 292bde6 to a237a15 Compare May 15, 2025 11:40
@MrToph
Copy link

MrToph commented May 16, 2025

it's because cargo install ignores the Cargo.lock by default.

This worked for me and should be the "official" command IMO:

cargo install --path . --locked

@stiefn
Copy link
Contributor Author

stiefn commented May 16, 2025

it's because cargo install ignores the Cargo.lock by default.

This worked for me and should be the "official" command IMO:

cargo install --path . --locked

Awesome, this works. Thanks for your input!

@stiefn stiefn requested a review from ritzdorf May 16, 2025 07:31
@stiefn stiefn force-pushed the change_installation_method branch from 29042e6 to f939798 Compare May 16, 2025 08:16
@ritzdorf
Copy link
Contributor

Thanks @MrToph

@ritzdorf ritzdorf merged commit 4f939be into main May 16, 2025
1 check 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.

3 participants