Skip to content

Commit

Permalink
Update comment about lsof being broken on BusyBox/musl.
Browse files Browse the repository at this point in the history
  • Loading branch information
jamadden committed Dec 11, 2021
1 parent cedd840 commit f1eebef
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/gevent/testing/openfiles.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,9 @@ def default_get_open_files(pipes=False, **_kwargs):
# Skip header and blank lines
continue
split = re.split(r'\s+', line)
# Note that this needs the real lsof; it won't work with
# the lsof that comes from BusyBox. You'll get parsing errors
# here.
_command, _pid, _user, fd = split[:4]
# Pipes (on OS X, at least) get an fd like "3" while normal files get an fd like "1u"
if fd[:-1].isdigit() or fd.isdigit():
Expand Down

0 comments on commit f1eebef

Please sign in to comment.