josiahcarlson / jobs Goto Github PK

Deprecation warnings are raised due to invalid escape sequences. This can be fixed by using raw strings or escaping the literals.

find . -iname '*.py' | grep -v example | xargs -P 4 -I{} python3.8 -Wall -m py_compile {}./jobs.py:282: DeprecationWarning: invalid escape sequence \d
  TS_RE = re.compile('^(\d{9,}(?:[.]\d*)?)$')
./jobs.py:283: DeprecationWarning: invalid escape sequence \d
  DT_RE = re.compile('^(\d{4})-(\d{2})-(\d{2})(?: (\d{2}):(\d{2})(?::(\d{2})(?:[.](?:\d+)?)?)?)?$')

I've seen some silent errors recently and after some shotgun debugging I ended up with the below minimal example:

import os
import signal
import threading
import time
#import jobs
def selfkill():
  time.sleep(5)
  os.kill(os.getpid(), signal.SIGTERM)
x = threading.Thread(target=selfkill)
x.start()
time.sleep(10)
print('OK')    

This will of course stop after 5 seconds with exit code ~143 or so, but if commenting out import jobs, it will exit without an error and exic code 0 :thinking_face:

I suspect this is happening because of https://github.com/josiahcarlson/jobs/blob/master/jobs.py#L323

I am not quite sure about the background and why we need to override the signal handler -- can you please help me understand what I am doing wrong or some pointers on how to fix this? In short, I want the above job (after uncommenting import jobs) to fail due to the SIGTERM.