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.

Sign up for GitHub

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

Jump to bottom

Display the target directory in cargo metadata output #4017

Closed
sfackler opened this issue May 10, 2017 · 3 comments · Fixed by #4022
Closed

Display the target directory in cargo metadata output #4017

sfackler opened this issue May 10, 2017 · 3 comments · Fixed by #4022
Labels
E-easy Experience: Easy

Comments

@sfackler
Copy link
Member

No description provided.

@alexcrichton
Copy link
Member

cc @matklad

@matklad matklad added the E-easy Experience: Easy label May 10, 2017
@matklad
Copy link
Member

matklad commented May 10, 2017

Should definitely do this! We even need it in IntelliJ Rust. I can implement this, but I don't have a ton of time right now. But it should be pretty strait forward to add: add target_dir to ExportInfo here and populate it from ws.target_dir().

@sfackler
Copy link
Member Author

Done!

@sfackler sfackler mentioned this issue May 10, 2017
Merged
sfackler added a commit to sfackler/cargo that referenced this issue May 10, 2017
@sfackler
Expose the target directory in cargo metadata
a367875 
Closes rust-lang#4017
bors added a commit that referenced this issue May 10, 2017
@bors
Auto merge of #4022 - sfackler:metadata-target-dir, r=matklad
7bbc807 
Expose the target directory in cargo metadata

Closes #4017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-easy Experience: Easy
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Expose the target directory in cargo metadata sfackler/cargo
3 participants
@alexcrichton @sfackler @matklad