Compare commits

..

No commits in common. "main" and "v3.1.1" have entirely different histories.
main ... v3.1.1

View File

@ -465,12 +465,9 @@ def main():
stdout_logger = proc_logger.getChild("stdout") stdout_logger = proc_logger.getChild("stdout")
stderr_logger = proc_logger.getChild("stderr") stderr_logger = proc_logger.getChild("stderr")
stdin_fd = proc.stdin.fileno() os.set_blocking(proc.stdin.fileno(), False)
stdout_fd = proc.stdout.fileno() os.set_blocking(proc.stdout.fileno(), False)
stderr_fd = proc.stderr.fileno() os.set_blocking(proc.stderr.fileno(), False)
os.set_blocking(stdin_fd, False)
os.set_blocking(stdout_fd, False)
os.set_blocking(stderr_fd, False)
poll = select.poll() poll = select.poll()
poll.register( poll.register(
@ -484,7 +481,7 @@ def main():
proc.stderr, proc.stderr,
select.POLLIN | select.POLLPRI | select.POLLHUP | select.POLLERR, select.POLLIN | select.POLLPRI | select.POLLHUP | select.POLLERR,
) )
pollc = 2 pollc = 3
# logger.debug("First poll...") # logger.debug("First poll...")
events = poll.poll() events = poll.poll()
# logger.debug("Done, %d event(s)", len(events)) # logger.debug("Done, %d event(s)", len(events))
@ -496,7 +493,7 @@ def main():
for rfd, event in events: for rfd, event in events:
# logger.debug("rfd=%d, event=%x", rfd, event) # logger.debug("rfd=%d, event=%x", rfd, event)
if event & select.POLLOUT: if event & select.POLLOUT:
if rfd == stdin_fd: if rfd == proc.stdin.fileno():
if chunk := pipe_buffer[:PIPE_BUF]: if chunk := pipe_buffer[:PIPE_BUF]:
# logger.debug( # logger.debug(
# "Writing chunk of length %d...", len(chunk) # "Writing chunk of length %d...", len(chunk)
@ -513,7 +510,7 @@ def main():
else: else:
proc.stdin.close() proc.stdin.close()
if event & select.POLLIN or event & select.POLLPRI: if event & select.POLLIN or event & select.POLLPRI:
if rfd == stdout_fd: if rfd == proc.stdout.fileno():
# logger.debug("Reading from stdout...") # logger.debug("Reading from stdout...")
if chunk := proc.stdout.buffer.read(PIPE_BUF): if chunk := proc.stdout.buffer.read(PIPE_BUF):
# logger.debug("Done, length %d", len(chunk)) # logger.debug("Done, length %d", len(chunk))
@ -530,12 +527,12 @@ def main():
# logger.debug("Split: %r, %r, %r", line, sep, rest) # logger.debug("Split: %r, %r, %r", line, sep, rest)
if sep: if sep:
stdout_logger.info( stdout_logger.info(
line.decode(errors="replace").rstrip() line.decode(errors="replace").strip()
) )
stdout_line_buffer = rest stdout_line_buffer = rest
else: else:
break break
if rfd == stderr_fd: if rfd == proc.stderr.fileno():
# logger.debug("Reading from stderr...") # logger.debug("Reading from stderr...")
if chunk := proc.stderr.buffer.read(PIPE_BUF): if chunk := proc.stderr.buffer.read(PIPE_BUF):
# logger.debug("Done, length %d", len(chunk)) # logger.debug("Done, length %d", len(chunk))
@ -552,28 +549,27 @@ def main():
# logger.debug("Split: %r, %r, %r", line, sep, rest) # logger.debug("Split: %r, %r, %r", line, sep, rest)
if sep: if sep:
stderr_logger.info( stderr_logger.info(
line.decode(errors="replace").rstrip() line.decode(errors="replace").strip()
) )
stderr_line_buffer = rest stderr_line_buffer = rest
else: else:
break break
if event & select.POLLERR: if event & select.POLLERR:
if rfd == stdin_fd: if rfd == proc.stdin.fileno():
logger.error("STDIN error") logger.error("STDIN error")
if rfd == stdout_fd: if rfd == proc.stdout.fileno():
logger.error("STDOUT error") logger.error("STDOUT error")
if rfd == stderr_fd: if rfd == proc.stderr.fileno():
logger.error("STDERR error") logger.error("STDERR error")
if event & select.POLLHUP: if event & select.POLLHUP:
if rfd == stdin_fd: if rfd == proc.stdin.fileno():
logger.debug("STDIN closed") logger.debug("STDIN closed")
if rfd == stdout_fd: if rfd == proc.stdout.fileno():
logger.debug("STDOUT closed") logger.debug("STDOUT closed")
pollc -= 1 if rfd == proc.stderr.fileno():
if rfd == stderr_fd:
logger.debug("STDERR closed") logger.debug("STDERR closed")
pollc -= 1
poll.unregister(rfd) poll.unregister(rfd)
pollc -= 1
if pollc > 0: if pollc > 0:
# logger.debug("Poll...") # logger.debug("Poll...")