search

cmyr / cargo-instruments

A cargo plugin to generate Xcode Instruments trace files
MIT License
681 stars 34 forks source link

fix: Update dependencies to get build working + fix Cargo manifest parse error #58

Closed afnanenayet closed 2 years ago

afnanenayet commented 2 years ago

This updates dependencies to the latest versions to fix an error with the binary being unable to parse a Cargo manifest.

This fixes an error I had where I would run cargo instruments ... and I would get an error

failed to parse manifest at `~/dev/diffsitter/Cargo.toml`

This also fixes an error I had when trying to build locally where the linking stage would fail.

cmyr commented 2 years ago

Thanks!

I'd like to understand better what wasn't working, can you provide an example of a Cargo.toml that was giving you problems, or any other info?

afnanenayet commented 2 years ago

Yeah sure, this is the manifest: https://github.com/afnanenayet/diffsitter/blob/07855f192e8ab9dfd84957022a53e394bf79b365/Cargo.toml

To replicate:

❯ cargo instruments -t time --release -- src/main.rs src/parse.rs
failed to parse manifest at `/Users/afnan/dev/diffsitter/Cargo.toml`
grtlr commented 2 years ago

I'm running into the same problem using workspaces. Here is the top-level Cargo.toml: https://github.com/iotaledger/bee/blob/mainnet-develop/Cargo.toml