squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Merge pull request #1028 from oli-obk/manifest_path

Browse source 
fix cargo clippy when using with `--manifest-path`
This commit is contained in:
Manish Goregaokar 2016-06-21 10:45:39 +01:00 committed by GitHub
commit 6ee6fe5776
2 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
clippy_lints/src/utils/cargo.rs
View file

@ -65,8 +65,13 @@ impl From<json::DecoderError> for Error {
}
}
pub fn metadata() -> Result<Metadata, Error> {
let output = Command::new("cargo").args(&["metadata", "--no-deps"]).output()?;
pub fn metadata(manifest_path: Option<String>) -> Result<Metadata, Error> {
let mut cmd = Command::new("cargo");
cmd.arg("metadata").arg("--no-deps");
if let Some(ref mani) = manifest_path {
cmd.arg(mani);
}
let output = cmd.output()?;
let stdout = from_utf8(&output.stdout)?;
Ok(json::decode(stdout)?)
}

3
src/main.rs
View file

@ -129,7 +129,8 @@ pub fn main() {
};
if let Some("clippy") = std::env::args().nth(1).as_ref().map(AsRef::as_ref) {
let mut metadata = cargo::metadata().expect("could not obtain cargo metadata");
let manifest_path = std::env::args().skip(2).find(|val| val.starts_with("--manifest-path="));
let mut metadata = cargo::metadata(manifest_path).expect("could not obtain cargo metadata");
assert_eq!(metadata.version, 1);
for target in metadata.packages.remove(0).targets {
let args = std::env::args().skip(2);