Add more instructions to error message

rust-lang · May 29, 2018 · 53480ac · 53480ac
1 parent e31f3c3
commit 53480ac
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/src/cargo/ops/cargo_package.rs b/src/cargo/ops/cargo_package.rs
@@ -360,8 +360,9 @@ fn run_verify(ws: &Workspace, tar: &FileLock, opts: &PackageOpts) -> CargoResult
         bail!(
             "Source directory was modified by build.rs during cargo publish. \
              Build scripts should not modify anything outside of OUT_DIR. \
-             Modified file: {}",
-             path.display()
+             Modified file: {}\n\n\
+             To proceed despite this, pass the `--no-verify` flag.",
+            path.display()
         )
     }
 

diff --git a/tests/testsuite/package.rs b/tests/testsuite/package.rs
@@ -1442,7 +1442,17 @@ fn do_not_package_if_src_was_modified() {
 
     assert_that(
         p.cargo("package"),
-        execs().with_status(101),
+        execs().with_status(101)
+               .with_stderr_contains(
+                   "\
+error: failed to verify package tarball
+
+Caused by:
+  Source directory was modified by build.rs during cargo publish. \
+Build scripts should not modify anything outside of OUT_DIR. Modified file: [..]src/generated.txt
+
+To proceed despite this, pass the `--no-verify` flag.",
+               ),
     );
 
     assert_that(