fixup! fixup! fixup! Move local imports to module level in sampling profiler

Author: pablogsal

Lib/profiling/sampling/cli.py

@@ -35,7 +35,11 @@
     SORT_MODE_NSAMPLES_CUMUL,
 )
 
-from ._child_monitor import ChildProcessMonitor
+try:
+    from ._child_monitor import ChildProcessMonitor
+except ImportError:
+    # _remote_debugging module not available on this platform (e.g., WASI)
+    ChildProcessMonitor = None
 
 try:
     from .live_collector import LiveStatsCollector
@@ -654,6 +658,11 @@ def _validate_args(args, parser):
 
     # --subprocesses is incompatible with --live
     if hasattr(args, 'subprocesses') and args.subprocesses:
+        if ChildProcessMonitor is None:
+            parser.error(
+                "--subprocesses is not available on this platform "
+                "(requires _remote_debugging module)."
+            )
         if hasattr(args, 'live') and args.live:
             parser.error("--subprocesses is incompatible with --live mode.")
 

Lib/test/test_profiling/test_sampling_profiler/test_children.py

@@ -35,13 +35,22 @@
     is_python_process = None
     _MAX_CHILD_PROFILERS = None
     _CLEANUP_INTERVAL_CYCLES = None
-from profiling.sampling.cli import (
-    _add_sampling_options,
-    _validate_args,
-    _build_child_profiler_args,
-    _build_output_pattern,
-    _setup_child_monitor,
-)
+
+try:
+    from profiling.sampling.cli import (
+        _add_sampling_options,
+        _validate_args,
+        _build_child_profiler_args,
+        _build_output_pattern,
+        _setup_child_monitor,
+    )
+except ImportError:
+    # cli module imports sample module which requires _remote_debugging
+    _add_sampling_options = None
+    _validate_args = None
+    _build_child_profiler_args = None
+    _build_output_pattern = None
+    _setup_child_monitor = None
 
 from .helpers import _cleanup_process