Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python 3.8 and get_pid : failure #84

Closed
SnarkBoojum opened this issue Nov 20, 2020 · 1 comment · Fixed by #209
Closed

Python 3.8 and get_pid : failure #84

SnarkBoojum opened this issue Nov 20, 2020 · 1 comment · Fixed by #209

Comments

@SnarkBoojum
Copy link

The terminado package for Debian fails with Python 3.8.6 ; you'll see here the full log, but here is the end:

ERROR: test_max_terminals (terminado.tests.basic_test.NamedTermTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/tornado/testing.py", line 98, in __call__
    result = self.orig_method(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/tornado/testing.py", line 594, in post_coroutine
    return self.io_loop.run_sync(
  File "/usr/lib/python3/dist-packages/tornado/ioloop.py", line 532, in run_sync
    return future_cell[0].result()
  File "/usr/lib/python3/dist-packages/tornado/gen.py", line 742, in run
    yielded = self.gen.throw(*exc_info)  # type: ignore
  File "/tmp/autopkgtest-lxc.lx8y8pgy/downtmp/build.JHO/src/terminado/tests/basic_test.py", line 216, in test_max_terminals
    pids = yield self.get_pids(tms)
  File "/usr/lib/python3/dist-packages/tornado/gen.py", line 735, in run
    value = future.result()
  File "/usr/lib/python3/dist-packages/tornado/gen.py", line 742, in run
    yielded = self.gen.throw(*exc_info)  # type: ignore
  File "/tmp/autopkgtest-lxc.lx8y8pgy/downtmp/build.JHO/src/terminado/tests/basic_test.py", line 127, in get_pids
    pid = yield tm.get_pid()
  File "/usr/lib/python3/dist-packages/tornado/gen.py", line 735, in run
    value = future.result()
  File "/usr/lib/python3/dist-packages/tornado/gen.py", line 748, in run
    yielded = self.gen.send(value)
  File "/tmp/autopkgtest-lxc.lx8y8pgy/downtmp/build.JHO/src/terminado/tests/basic_test.py", line 98, in get_pid
    pid = int(stdout.split('\n')[1])
ValueError: invalid literal for int() with base 10: '\x1b[?2004l\r3257\r'

Looking at that function get_pid, it looks like it expects something special when os.name=='nt', and something quite trivial otherwise, but that's not what it actually gets. I'll try to dig around to understand the issue better.

@SnarkBoojum
Copy link
Author

I got around the error patching trivially, but it's obviously not a proper fix :
fix_975241.txt

blink1073 pushed a commit that referenced this issue Dec 2, 2022
Co-authored-by: Steven Silvester <steven.silvester@ieee.org>
Fixes #84
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant