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

Support -p/--package argument #56

Open
codyps opened this issue Dec 5, 2021 · 1 comment
Open

Support -p/--package argument #56

codyps opened this issue Dec 5, 2021 · 1 comment

Comments

@codyps
Copy link
Contributor

codyps commented Dec 5, 2021

Right now, I have to cd into a particular package within my workspace to use cargo-instruments. Other builtin cargo commands (like cargo run and cargo build) support a -p/--package argument so that one doesn't need to cd around. It would be good for cargo-instruments to support the same option.

@cmyr
Copy link
Owner

cmyr commented Dec 6, 2021

Yes, this would be nice. I actually hacked on this for a while, and it ended up not being trivial, so I didn't follow up; basically to do this correctly we would want to pretty closely match the behaviour of the built-in cargo tools, which means doing this like respecting the default-run and default-members manifest fields, and doing this requires a fairly large-scale rearchitecture. 😒

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

No branches or pull requests

2 participants