about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSteven Fackler <sfackler@gmail.com>2013-09-28 22:03:54 -0700
committerSteven Fackler <sfackler@gmail.com>2013-09-28 22:12:24 -0700
commitc685e0879c485f3ff44bf46aeaf24f114bf9d097 (patch)
treef01abead1aff6eacea4471ff56a92ce178000b32
parent92e7bb67a8038ff73ad9dc88d8be3d3df4777282 (diff)
downloadrust-c685e0879c485f3ff44bf46aeaf24f114bf9d097.tar.gz
rust-c685e0879c485f3ff44bf46aeaf24f114bf9d097.zip
Don't redirect output in rustpkg test
Previously, if tests failed, you'd only get stderr which isn't very
useful, especially if the failure didn't happen directly in a test
function (e.g None.unwrap()).
-rw-r--r--src/librustpkg/rustpkg.rs10
1 files changed, 2 insertions, 8 deletions
diff --git a/src/librustpkg/rustpkg.rs b/src/librustpkg/rustpkg.rs
index 76edd7feff7..e4f88081239 100644
--- a/src/librustpkg/rustpkg.rs
+++ b/src/librustpkg/rustpkg.rs
@@ -620,14 +620,8 @@ impl CtxMethods for BuildContext {
         match built_test_in_workspace(pkgid, workspace) {
             Some(test_exec) => {
                 debug!("test: test_exec = %s", test_exec.to_str());
-                let p_output = run::process_output(test_exec.to_str(), [~"--test"]);
-                if p_output.status == 0 {
-                    println(str::from_utf8(p_output.output));
-                }
-                    else {
-                    println(str::from_utf8(p_output.error));
-                }
-                os::set_exit_status(p_output.status);
+                let status = run::process_status(test_exec.to_str(), [~"--test"]);
+                os::set_exit_status(status);
             }
             None => {
                 error(fmt!("Internal error: test executable for package ID %s in workspace %s \