comparison mercurial/worker.py @ 40472:03f7d0822ec1 stable

worker: do not swallow exception occurred in main process

Before, SystemExit(255) would be most likely raised since the worker processes
were terminated by the main process and the status would be set to 255 in
response. We should instead re-raise the exception occurred first. It's pretty
hard to debug problems like the issue 6035 with no traceback.

40471:bb5d74a35477

    def cleanup():
        signal.signal(signal.SIGINT, oldhandler)
        waitforworkers()
        signal.signal(signal.SIGCHLD, oldchldhandler)
        selector.close()
        status = problem[0]
        if status:
            if status < 0:
                os.kill(os.getpid(), -status)
            sys.exit(status)
    try:
        openpipes = len(pipes)
        while openpipes > 0:
            for key, events in selector.select():
                try:

                    raise
    except: # re-raises
        killworkers()
        cleanup()
        raise
    cleanup()

def _posixexitstatus(code):
    '''convert a posix exit status into the same form returned by
    os.spawnv

40472:03f7d0822ec1

    def cleanup():
        signal.signal(signal.SIGINT, oldhandler)
        waitforworkers()
        signal.signal(signal.SIGCHLD, oldchldhandler)
        selector.close()
        return problem[0]
    try:
        openpipes = len(pipes)
        while openpipes > 0:
            for key, events in selector.select():
                try:

                    raise
    except: # re-raises
        killworkers()
        cleanup()
        raise
    status = cleanup()
    if status:
        if status < 0:
            os.kill(os.getpid(), -status)
        sys.exit(status)

def _posixexitstatus(code):
    '''convert a posix exit status into the same form returned by
    os.spawnv