Skip to content

Add compiler cargo to ftplugin/rust.vim? #73

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

Closed
ticki opened this issue Feb 21, 2016 · 8 comments
Closed

Add compiler cargo to ftplugin/rust.vim? #73

ticki opened this issue Feb 21, 2016 · 8 comments
Assignees

Comments

@ticki
Copy link

ticki commented Feb 21, 2016

So that you can do :make and :cc for populating the QuickFix list.

@ticki ticki changed the title Add compiler cargo to ftplugin/rust.vim? Add compiler cargo to ftplugin/rust.vim? Feb 21, 2016
@joshtriplett
Copy link
Member

This shouldn't happen unconditionally, only if searching upwards finds a Cargo.toml file.

@mbyio
Copy link
Contributor

mbyio commented Jul 29, 2016

I think this is already implemented, you just have to run :compiler cargo first, then you can do :make build. (It's not documented for some reason.) They also have :compiler rustc. I assume it doesn't automatically set it up so that you're able to choose which one you want.

@joshtriplett
Copy link
Member

@michael-younkin It'd be nice to automatically look for a Cargo.toml file, enable :compiler cargo if so, and enable :compiler rustc if not.

@somini
Copy link

somini commented Aug 26, 2016

It's easy to search for a "Cargo.toml" file in parent directories.
#76 (comment)

@Victor-Savu
Copy link
Contributor

I'm with @joshtriplett . At least some sane default should exist, since no action means that GNU Make will be used by default, which is almost never what the user wants.

@sssilver
Copy link

sssilver commented Feb 8, 2018

Please, can we have this?

@Victor-Savu
Copy link
Contributor

@sssilver for what it's worth, we can at least try 😄

@da-x da-x self-assigned this Jun 30, 2018
@da-x
Copy link
Member

da-x commented Jul 5, 2018

PR was merged.

@da-x da-x closed this as completed Jul 5, 2018
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

7 participants