diff options
| author | Steven Fackler <sfackler@gmail.com> | 2013-09-28 22:03:54 -0700 |
|---|---|---|
| committer | Steven Fackler <sfackler@gmail.com> | 2013-09-28 22:12:24 -0700 |
| commit | c685e0879c485f3ff44bf46aeaf24f114bf9d097 (patch) | |
| tree | f01abead1aff6eacea4471ff56a92ce178000b32 | |
| parent | 92e7bb67a8038ff73ad9dc88d8be3d3df4777282 (diff) | |
| download | rust-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.rs | 10 |
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 \ |
