fix: properly mark the C extension as safe for free-threading. #1799

https://github.com/python/cpython/blob/main/Doc/howto/free-threading-extensions.rst#c-api-extension-support-for-free-threading
nedbat · Jun 20, 2024 · b18485e · b18485e
1 parent f7b71e7
commit b18485e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/python-nightly.yml b/.github/workflows/python-nightly.yml
@@ -86,11 +86,6 @@ jobs:
       - name: "Check out the repo"
         uses: "actions/checkout@v4"
 
-      - name: "Set PYTHON_GIL"
-        if: "${{ matrix.nogil }}"
-        run: |
-          echo "PYTHON_GIL=0" >> $GITHUB_ENV
-
       - name: "Install ${{ matrix.python-version }} with deadsnakes"
         uses: deadsnakes/action@6c8b9b82fe0b4344f4b98f2775fcc395df45e494
         if: "!startsWith(matrix.python-version, 'pypy-')"

diff --git a/coverage/ctracer/module.c b/coverage/ctracer/module.c
@@ -31,6 +31,10 @@ PyInit_tracer(void)
         return NULL;
     }
 
+#ifdef Py_GIL_DISABLED
+    PyUnstable_Module_SetGIL(mod, Py_MOD_GIL_NOT_USED);
+#endif
+
     if (CTracer_intern_strings() < 0) {
         return NULL;
     }