Made adjustments based on feedback from review.
authorPatrik Svensson <patrik@patriksvensson.se>
Sat, 5 May 2018 14:17:59 +0000 (16:17 +0200)
committerPatrik Svensson <patrik@patriksvensson.se>
Sat, 5 May 2018 14:20:53 +0000 (16:20 +0200)
src/cargo/ops/cargo_run.rs
tests/testsuite/run.rs

index 7309b394d6ae6658d0ee761a563b06e303e6eac9..a2e35c3d53e2afdc726bf580a2bf1cda0135ee4b 100644 (file)
@@ -26,7 +26,7 @@ pub fn run(
         },
     };
 
-let bins: Vec<_> = pkg.manifest()
+    let bins: Vec<_> = pkg.manifest()
         .targets()
         .iter()
         .filter(|a| {
@@ -48,12 +48,12 @@ let bins: Vec<_> = pkg.manifest()
     }
 
     if bins.len() == 1 {
-        let bin = bins.first().unwrap();
-        match *bin.1 {
-            TargetKind::ExampleLib(..) => { 
+        let &(name, kind) = bins.first().unwrap();
+        match kind {
+            &TargetKind::ExampleLib(..) => { 
                 bail!(
-                    "example target `{}` is a library and cannot be executed.",
-                    bin.0
+                    "example target `{}` is a library and cannot be executed",
+                    name
                 ) 
             },
             _ => { }
@@ -62,9 +62,7 @@ let bins: Vec<_> = pkg.manifest()
 
     if bins.len() > 1 {
         if !options.filter.is_specific() {
-
-            let names : Vec<&str> = bins.into_iter().map(|bin| bin.0).collect();
-
+            let names: Vec<&str> = bins.into_iter().map(|bin| bin.0).collect();
             bail!(
                 "`cargo run` requires that a project only have one \
                  executable; use the `--bin` option to specify which one \
index bf2e37fe2eead695721ba826621a6c54baad2c40..3744aa036a55ca7a39e2104700032c74b2b850c0 100644 (file)
@@ -415,7 +415,7 @@ fn run_library_example() {
         p.cargo("run").arg("--example").arg("bar"),
         execs()
             .with_status(101)
-            .with_stderr("[ERROR] example target `bar` is a library and cannot be executed."),
+            .with_stderr("[ERROR] example target `bar` is a library and cannot be executed"),
     );
 }