rosotacom-dev 2.5.dev69__tar.gz → 2.5.dev70__tar.gz

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rosotacom-dev
-Version: 2.5.dev69
+Version: 2.5.dev70
 Summary: Host CLI for ROS OTA communication sessions.
 Author: develNor
 License-Expression: BSD-3-Clause

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/docs/ota-pcap.md

@@ -140,6 +140,13 @@ thread, so a capture that misbehaves cannot reach the snapshot an operator is
 watching. It is stopped with `SIGINT` and waited for, because that is what
 writes the sidecar JSON.
 
+It is launched by **file path**, not as `python3 -m com_py.ota_pcap`. sudo
+resets the environment, so `PYTHONPATH` does not survive it and the package is
+not importable on the other side — which is what a two-host run answered,
+`No module named 'com_py'`, once the permission problem was out of the way.
+`ota_pcap.py` imports nothing but the standard library, so a path needs no
+package at all, and a test pins that it stays that way.
+
 ## Cost
 
 Header-only at 96 bytes: about 6 MB per minute at 500 packets/s, so a two-hour

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/src/rosotacom/resources/ws/ros2src/com_py/com_py/status_overview.py

@@ -58,6 +58,7 @@ import signal
 import subprocess
 import sys
 import threading
+from pathlib import Path
 from typing import Dict, Optional
 
 import yaml
@@ -558,8 +559,21 @@ class StatusOverview(Node):
             return
 
         path = os.path.join(output_dir, "ota.pcap")
+        # By file path, not `-m com_py.ota_pcap`. sudo resets the environment,
+        # so PYTHONPATH does not survive and the package is not importable on
+        # the other side of it — a two-host run answered exactly that,
+        # "No module named 'com_py'", after the permission problem was fixed.
+        # `ota_pcap.py` imports nothing but the standard library (a test pins
+        # that), so a path needs no package at all.
+        script = Path(__file__).resolve().with_name("ota_pcap.py")
+        if not script.is_file():
+            self.get_logger().warning(
+                f"status_overview: ota_pcap is enabled but {script} is not here; "
+                "capturing nothing"
+            )
+            return
         argv = [
-            sys.executable, "-m", "com_py.ota_pcap",
+            sys.executable, str(script),
             "--iface", interface,
             "--out", path,
             "--snaplen", str(int(self.get_parameter("ota_pcap_snaplen").value)),

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/src/rosotacom_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rosotacom-dev
-Version: 2.5.dev69
+Version: 2.5.dev70
 Summary: Host CLI for ROS OTA communication sessions.
 Author: develNor
 License-Expression: BSD-3-Clause

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/src/rosotacom_dev.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
   "tag": "2.4",
-  "distance": 69,
-  "node": "g429a2536fa56d96c76f3c306809f3dc6864f051e",
+  "distance": 70,
+  "node": "g2a4d43769fad290b84ccf9e3a25455c6e45ce093",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-21"

{rosotacom_dev-2.5.dev69 → rosotacom_dev-2.5.dev70}/tests/unit/test_ota_pcap.py

@@ -345,6 +345,10 @@ def test_the_node_launches_it_through_sudo_and_hands_root_straight_back() -> Non
     source because the node needs rclpy, which the host suite does not have."""
     source = (WS_PY / "com_py" / "status_overview.py").read_text(encoding="utf-8")
     assert 'argv = ["sudo", "-n", *argv]' in source
+    # By path, never `-m`: sudo resets the environment, so PYTHONPATH does not
+    # reach the other side of it and the package is not importable there.
+    assert '"-m", "com_py.ota_pcap"' not in source
+    assert 'Path(__file__).resolve().with_name("ota_pcap.py")' in source
     assert '"--drop-to", f"{os.getuid()}:{os.getgid()}"' in source
     # And the stop goes the same way: the child is root, so an ordinary kill
     # would be refused — and it must be SIGINT, which is what writes ota.json.