Merge #82

82: Filter "custom-build" artifact kinds from automatching r=adamgreig a=therealprof

If a project uses build.rs, cargo will add a "custom-build" artifact
kind to the build which is confusing information because we can only use
concrete compiled "bin" or "example" binaries for binutils, so let's
only admit real "bin" or "example" kinds into the auto matcher.

Fixes #81

Signed-off-by: Daniel Egger <daniel@eggers-club.de>

Co-authored-by: Daniel Egger <daniel@eggers-club.de>

Author: bors[bot]

src/lib.rs

@@ -120,7 +120,6 @@ enum BuildType<'a> {
 impl<'a> BuildType<'a> {
     fn matches(&self, artifact: &Artifact) -> bool {
         match self {
-            BuildType::Any => true,
             BuildType::Bin(target_name)
             | BuildType::Example(target_name)
             | BuildType::Test(target_name)
@@ -129,6 +128,13 @@ impl<'a> BuildType<'a> {
             }
             // For info about 'kind' values see:
             // https://github.com/rust-lang/cargo/blob/d47a9545db81fe6d7e6c542bc8154f09d0e6c788/src/cargo/core/manifest.rs#L166-L181
+            // The only "Any" artifacts we can support are bins and examples, so let's make sure
+            // no-one slips us a "custom-build" in form of a build.rs in
+            BuildType::Any => artifact
+                .target
+                .kind
+                .iter()
+                .any(|s| s == "bin" || s == "example"),
             // Since LibKind can be an arbitrary string `LibKind:Other(String)` we filter by what it can't be
             BuildType::Lib => artifact.target.kind.iter().any(|s| {
                 s != "bin" && s != "example" && s != "test" && s != "custom-build" && s != "bench"