Include a Cargo.lock

[arduano]

Hi! I was just attempting to make a Nix package based on this repository, and doing that requires having a Cargo.lock file in the repository (or manually providing one myself, which I'll do for now). It's generally standard practice to include a Cargo.lock, though correct me if the unique requirements of this project mean that one can't be included.

[tyt2y3]

You're right, I think we simply git ignored our Cargo.lock. We'd include one from the next release onwards.

[tyt2y3]

You are right, I messed things up, check the Cargo.lock at the root 13785e4

[arduano]

Perfect, probably just needs updating the building.md file. I was slightly wrong in my above comment, where it's not cargo build --project=indexer, instead it's cargo build --project=firedbg-stream-indexer or any other package name (instead of folder name). Or cargo build --all also should probably work. Though I'm also slightly confused why --release isn't included there.

[arduano]

Also, annoyingly, I got it building fine on NixOS, but the extension still wouldn't detect the installed binaries. If you don't mind, could you please point me to the extension code that checks if the binaries exist? I couldn't find that code for some reason.

[tyt2y3]

The vscode extension is currently not open source, but there is an option to set the path for the binaries

If unset, it defaults to ~/.cargo/bin

[arduano]

Ah I see, will keep that in mind, thanks!