[GitHub] [qpid-dispatch] kgiusti commented on a change in pull request #1153: DO NOT MERGE - extend travis timeouts hugely

[GitBox <gi...@apache.org>]

kgiusti commented on a change in pull request #1153:
URL: https://github.com/apache/qpid-dispatch/pull/1153#discussion_r618456683



##########
File path: .travis.yml
##########
@@ -204,7 +212,7 @@ script:
 - popd
 # Workaround on macOS for PROTON-808 Binaries have their library locations stripped
 - if [[ "${OSTYPE}" == "darwin"* ]]; then install_name_tool -add_rpath $PREFIX/lib/. $PREFIX/lib/proton/bindings/python/_cproton.so; fi
-- ctest -V ${DISPATCH_CTEST_EXTRA} && if [ "$BUILD_TYPE" = "Coverage" ]; then cmake --build . --target coverage; fi
+- ctest --timeout 1200 -V ${DISPATCH_CTEST_EXTRA} && if [ "$BUILD_TYPE" = "Coverage" ]; then cmake --build . --target coverage; fi

Review comment:
       Good point.  The 300 second (5 minute) per-test timeout would trigger first I believe, since each test prints something at the start and end.
   




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org