Fix: build and runtime deps should be unique

Before this patch, if a package had the following dependencies:

    build = ["other =1"]
    runtime = ["other =1"]

the package "other" in version "1" was built twice.
This is not necessary, we only need to build "other" once. This patch
fixes this bug.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>

1 files changed, 4 insertions, 1 deletions

diff --git a/src/package/package.rs b/src/package/package.rs
index 4be822f..998816d 100644
--- a/src/package/package.rs
+++ b/src/package/package.rs
@@ -3,6 +3,7 @@ use std::path::PathBuf;
 
 use anyhow::Result;
 use getset::Getters;
+use itertools::Itertools;
 use serde::Deserialize;
 use serde::Serialize;
 
@@ -86,7 +87,9 @@ impl Package {
             .cloned()
             .map(|d| d.parse_as_name_and_version());
 
-        build_iter.chain(runtime_iter)
+        build_iter
+            .chain(runtime_iter)
+            .unique_by(|res| res.as_ref().ok().map(|tpl| tpl.clone()))
     }
 }