Repository: mesos Updated Branches: refs/heads/master 520b72985 -> 351bade6c
Changed failure check for a call to os::system(). The previous check for `None()` was missing the case where the program exited with a non-zero exit status. Review: https://reviews.apache.org/r/66776/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/bb8c9a28 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/bb8c9a28 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/bb8c9a28 Branch: refs/heads/master Commit: bb8c9a28418e1632c225f669f4d0326332d3a835 Parents: 520b729 Author: Benno Evers <[email protected]> Authored: Fri May 4 13:37:25 2018 +0200 Committer: Benjamin Bannier <[email protected]> Committed: Fri May 4 13:37:25 2018 +0200 ---------------------------------------------------------------------- 3rdparty/libprocess/src/memory_profiler.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/bb8c9a28/3rdparty/libprocess/src/memory_profiler.cpp ---------------------------------------------------------------------- diff --git a/3rdparty/libprocess/src/memory_profiler.cpp b/3rdparty/libprocess/src/memory_profiler.cpp index b765246..a4c6be6 100644 --- a/3rdparty/libprocess/src/memory_profiler.cpp +++ b/3rdparty/libprocess/src/memory_profiler.cpp @@ -322,7 +322,7 @@ Try<Nothing> generateJeprofFile( inputPath, outputPath).get()); - if (result.isNone()) { + if (result != 0) { return Error( "Error trying to run jeprof. Please make sure that jeprof is installed" " and that the input file contains data. For more information, please"
