diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2015-12-14 16:47:02 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2015-12-14 16:47:02 +0200 |
commit | d4eecda55cc543a5bf4f478d1a6d50a4c8a01ac4 (patch) | |
tree | 73a4d2100842455f767972b12d8f9d90f9a2e8e3 | |
parent | f81a767e262309d47d4a0d521a7a2c12f0abad99 (diff) |
Add note on redirecting STDERR in cli test
-rw-r--r-- | build/cli/module.cxx | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/build/cli/module.cxx b/build/cli/module.cxx index 67e2137..de7d576 100644 --- a/build/cli/module.cxx +++ b/build/cli/module.cxx @@ -138,6 +138,10 @@ namespace build } catch (const process_error& e) { + // In some cases this is not enough (e.g., the runtime linker + // will print scary errors if some shared libraries are not + // found. So it would be good to redirect child's STDERR. + // if (!optional) error << "unable to execute " << cli << ": " << e.what (); |