Add explanatory comments.

.github/workflows/windows.yml

@@ -231,7 +231,12 @@ jobs:
       with:
         python-version: '3.11.4'
 
-    # Without this, venv stopped working in 20240929.1 runner.
+    # virtualenv is no longer included as of 20240929.1 runner so
+    # install it ourselves. See
+    # https://github.com/actions/runner-images/issues/10749.
+    #
+    # Note that it is the builds of sdist and wheel that require virtualenv
+    # not the explicit dependencies of pyktx.
     - name: Install Python virtualenv
       if: matrix.options.py == 'ON' && matrix.check_mkvk != 'ONLY'
       run: pip install virtualenv

interface/python_binding/CMakeLists.txt

@@ -88,6 +88,8 @@ add_custom_command(
             LIBKTX_INCLUDE_DIR=${CMAKE_SOURCE_DIR}/include
             LIBKTX_LIB_DIR=${LIBKTX_LIB_DIR}
             LIBKTX_VERSION=${KTX_VERSION_NORMALIZED}
+        # Bulding sdist requires a virtual env. I have not found a
+        # a way via find_package to ensure venv support is installed.
         ${PYTHON_EXECUTABLE} -m build --sdist --outdir ${DIST_DIR}
     WORKING_DIRECTORY
         ${SOURCE_DIR}
@@ -103,6 +105,7 @@ add_custom_command(
             LIBKTX_INCLUDE_DIR=${CMAKE_SOURCE_DIR}/include
             LIBKTX_LIB_DIR=${LIBKTX_LIB_DIR}
             LIBKTX_VERSION=${KTX_VERSION_NORMALIZED}
+        # Ditto with sdist venv comment.
         ${PYTHON_EXECUTABLE} -m build --wheel --outdir ${DIST_DIR}
     WORKING_DIRECTORY
         ${SOURCE_DIR}