We need to cap the time it takes to install requirements, just like we do internally currently. To enable this, I changed the Subprocess class such that it accepts a timeout now, and sends a SIGTERM/SIGKILL to the process if it takes more than this.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available: N/A
Description of changes:
We need to cap the time it takes to install requirements, just like we do internally currently. To enable this, I changed the
Subprocess
class such that it accepts a timeout now, and sends a SIGTERM/SIGKILL to the process if it takes more than this.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.