Merge pull request #52 from renan-r-santos/fix-blocking-subprocess-call

Fix: check for OS before choosing blocking version of subprocess.run

Author: renan-r-santos

.github/workflows/ci.yml

@@ -12,7 +12,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        pixi-version: ['0.30.0', '0.41.4']
+        pixi-version: ['0.30.0', '0.45.0']
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout repo

package.json

@@ -1,6 +1,6 @@
 {
     "name": "pixi-kernel",
-    "version": "0.6.2",
+    "version": "0.6.3",
     "description": "Jupyter kernels using Pixi for reproducible notebooks.",
     "keywords": [
         "kernel",

pixi_kernel/async_subprocess.py

@@ -1,13 +1,14 @@
+import platform
 import subprocess
-from asyncio import SelectorEventLoop, create_subprocess_exec, get_event_loop
+from asyncio import SelectorEventLoop, create_subprocess_exec, get_running_loop
 from asyncio.subprocess import PIPE
 from typing import Any
 
 
 async def subprocess_exec(program: str, *args: str, **kwargs: Any) -> tuple[int, str, str]:
     # The SelectorEventLoop does not support asyncio.subprocess
     # https://github.com/renan-r-santos/pixi-kernel/issues/39
-    if isinstance(get_event_loop(), SelectorEventLoop):
+    if isinstance(get_running_loop(), SelectorEventLoop) and platform.system() == "Windows":
         result = subprocess.run([program, *args], capture_output=True, text=True, **kwargs)  # noqa: ASYNC221
         return result.returncode, result.stdout, result.stderr
     else:

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pixi-kernel"
-version = "0.6.2"
+version = "0.6.3"
 description = "Jupyter kernels using Pixi for reproducible notebooks"
 license = { text = "MIT" }
 authors = [