Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/master' into edge
Browse files Browse the repository at this point in the history
  • Loading branch information
jonasbardino committed Jan 29, 2025
2 parents f2fefa8 + 061d7bc commit 2db5a4c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions mig/shared/serial.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,10 @@
# seen in sshd+sftpsubsys sessions. Apparently there's a bug / C-API
# incompatibility when using yaml in embedded python interpreters,
# due to yaml using its own nested C-extensions. In practice it
# results in a metaclass conflict TypeError upon yaml re-init there
# like described in
# results in a metaclass conflict TypeError or SystemError upon yaml
# re-init there like described in
# https://github.com/ros-drivers/rosserial/issues/450
# On Rocky 9 it's the TypeError and on Rocky 8 the SystemError.
try:
import json
except ImportError:
Expand All @@ -49,7 +50,7 @@
import yaml
except ImportError:
yaml = None
except TypeError:
except (TypeError, SystemError):
# NOTE: this should not really happen but it does with sshd+sftpsubsys in
# our PAM module hooking into this code as described above. We don't
# actually need yaml in that case so just silently ignore it here and
Expand Down

0 comments on commit 2db5a4c

Please sign in to comment.