Use a few more chain_error calls

alexcrichton · alexcrichton · commit f00c223d3a5e · 2017-05-16T08:05:26.000-07:00
diff --git a/src/cargo/core/workspace.rs b/src/cargo/core/workspace.rs
@@ -8,7 +8,7 @@ use glob::glob;
 use core::{Package, VirtualManifest, EitherManifest, SourceId};
 use core::{PackageIdSpec, Dependency, Profile, Profiles};
 use ops;
-use util::{Config, CargoResult, Filesystem, human};
+use util::{Config, CargoResult, Filesystem, human, ChainError};
 use util::paths;
 
 /// The core abstraction in Cargo for working with a workspace of crates.
@@ -549,12 +549,12 @@ fn expand_member_path(path: &Path) -> CargoResult<Vec<PathBuf>> {
         Some(p) => p,
         None => return Ok(Vec::new()),
     };
-    let res = glob(path).map_err(|e| {
-        human(format!("could not parse pattern `{}`: {}", &path, e))
+    let res = glob(path).chain_error(|| {
+        human(format!("could not parse pattern `{}`", &path))
     })?;
     res.map(|p| {
-        p.or_else(|e| {
-            Err(human(format!("unable to match path to pattern `{}`: {}", &path, e)))
+        p.chain_error(|| {
+            human(format!("unable to match path to pattern `{}`", &path))
         })
     }).collect()
 }
diff --git a/src/cargo/util/errors.rs b/src/cargo/util/errors.rs
@@ -11,6 +11,7 @@ use core::TargetKind;
 
 use curl;
 use git2;
+use glob;
 use semver;
 use serde_json;
 use term;
@@ -370,6 +371,8 @@ from_error! {
     term::Error,
     num::ParseIntError,
     str::ParseBoolError,
+    glob::PatternError,
+    glob::GlobError,
 }
 
 impl From<string::ParseError> for Box<CargoError> {
@@ -401,6 +404,8 @@ impl CargoError for ffi::NulError {}
 impl CargoError for term::Error {}
 impl CargoError for num::ParseIntError {}
 impl CargoError for str::ParseBoolError {}
+impl CargoError for glob::PatternError {}
+impl CargoError for glob::GlobError {}
 
 // =============================================================================
 // Construction helpers
