Investigate how to make it work on Windows

[AntoineCezar]

Signals doesn't work on Windows.

Investigate for thread or process based alternative.

[AntoineCezar]

Working but dirty example:

import sys
from threading import Timer
import time
# import thread  # python 2
import _thread as thread  # python 3
from contextlib import contextmanager


@contextmanager
def timeout(t):
    try:
        yield Timer(t, thread.interrupt_main).start()
    except KeyboardInterrupt:
        raise TimeoutError


def forever():
    while 1:
        sys.stdout.write('.')
        sys.stdout.flush()
        time.sleep(1)


try:
    with timeout(3):
        forever()
except TimeoutError:
    print('timeout')

[AntoineCezar]

Added KeyboardInterrupt disambiguation:

import sys
from threading import Timer
import time
import _thread as thread
from contextlib import contextmanager


@contextmanager
def timeout(t):
    try:
        timer = Timer(t, thread.interrupt_main)
        timer.daemon = True
        yield timer.start()
    except KeyboardInterrupt:
        if timer.is_alive():
            raise
        raise TimeoutError


def forever():
    while 1:
        sys.stdout.write('.')
        sys.stdout.flush()
        time.sleep(1)


try:
    with timeout(3):
        forever()
except TimeoutError:
    print('timeout')

[AntoineCezar]

@vvscloud the above snippet might interrest you.

[vvscloud]

Hi AntoineCezar,

I wrote a sample snippet which works on windows (Tested it as well); Please validate from your end and share feedback if any.

Hope it helps.

def run(args, cwd=None, shell=False, kill_tree=True, timeout=-1, env=None):
    from threading import Timer
    p = subprocess.Popen(args, shell=shell, cwd=cwd, stdout=subprocess.PIPE,
                         stderr=subprocess.PIPE, env=env)
    if timeout != -1:
        timer = Timer(timeout, p.kill)
        timer.start()
    stdout, stderr = p.communicate()
    if timeout != -1:
        timer.cancel()
    return p.returncode, stdout, stderr

[AntoineCezar]

Hi @vvscloud,

I've tested your snippet and it works like a charm. I can't figure out how to make it work in timeoutcontext has it would require sending the context to a different process (probably with multiprocessing instead of subprocess). However it's a nice way to timeout shell processes.

Have you tried the thread based snippet? I do not have any Window for the moment. Your feedback would confirm that it's a valid option.

[AntoineCezar]

Working thread based solution that does not send your code to another thread: https://github.com/AntoineCezar/timeoutcontext/tree/thread_timeout

Before I release this, I must:

• Test it on Windows (that I don't have right now)
• Update the README
• Ask for experts whether my solution is not dangerous (who?)

[vvscloud]

I can test on windows. Please let me know.

[AntoineCezar]

@vvscloud you can test the thread_timeout branch it will automatically use threading on Windows