Skip to content

Commit

Permalink
Fix multithreading issue
Browse files Browse the repository at this point in the history
  • Loading branch information
jngrad committed Sep 22, 2018
1 parent eed9856 commit 7cac5fd
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion maintainer/CI/build_cmake.sh
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ if $make_check; then
done
fi
cmd "make -j${build_procs} check_unit_tests $make_params" || exit 1
cmd "make -j${build_procs} check_cmake_install $make_params" || exit 1
cmd "make check_cmake_install $make_params" || exit 1

end "TEST"
else
Expand Down
5 changes: 3 additions & 2 deletions testsuite/cmake/BashUnitTests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ function try_catch_silent() {
detect_open_mpi || return 1
(
set -e # exit from current subshell on first error
"$@" 1>/dev/null 2>/dev/null
"$@" 2>&1
)
return $?
}
Expand Down Expand Up @@ -395,12 +395,13 @@ function assert_zero() {
function assert_return_code() {
try_catch_silent "$@"
local retcode=$?
local log=$(try_catch_silent "$@")
local message="non-zero return code (${retcode}) for command \`$*\`"
if [ "${retcode}" -eq "0" ]
then
log_success
else
log_failure "${message}"
log_failure "${message}" "${log}" $(ls /builds/espressomd/espresso/build/testsuite/cmake/install//tmp/espresso-unit-tests/lib/python2.7/dist-packages/espressomd 2>&1)
fi
}

Expand Down

0 comments on commit 7cac5fd

Please sign in to comment.