fixup! Fix venv check by inverting exit code and adding more info

Author: JohT

.github/workflows/internal-check-python-venv-support.yml

@@ -32,19 +32,17 @@ jobs:
       env: 
         USE_VIRTUAL_PYTHON_ENVIRONMENT_VENV: "true"
       # 1.  Run the script under test to create, activate and install the virtual environment
-      # 2.  Start with an overall check with pip
-      # 3.  Run pip in dry-mode with full output first to get all infos and fail on missing packages.
-      # 4a. The leading exclamation mark tells the shell to invert the returned exit code. 
+      # 2.  Run pip install for the current environment in dry-mode with full output first to get all infos and fail on missing packages.
+      # 3a. The leading exclamation mark tells the shell to invert the returned exit code. 
       #     Failing because "Would install" is missing leads to a successful execution. 
-      # 5b. Run pip in dry-run mode without installing or resolving dependencies
-      # 5c. Suppress all pip output (stderr)
-      # 5d. Check if pip *would install* anything using grep
-      # 5e. If there are missing dependencies and the environment is incomplete, fail with code 1 (indicates all requirements already satisfied)
+      # 3b. Run pip install for the current environment in dry-run mode without installing or resolving dependencies
+      # 3c. Suppress all pip output (stderr)
+      # 3d. Check if pip would install anything using grep
+      # 3e. If there are missing dependencies and the environment is incomplete, 
+      #     fail with code 1 (indicates all requirements already satisfied)
       #
       # Output installed dependencies for troubleshooting purposes:.venv/bin/pip freeze
-      # TODO remove:
-      # pip install --dry-run --ignore-installed --requirement "./requirements.txt"
-      # ! pip install --dry-run --no-deps --no-cache-dir --no-index --requirement "./requirements.txt" 2>/dev/null | grep -q "Would install"
       run: |
         source ./scripts/activatePythonEnvironment.sh
-        .venv/bin/pip install --dry-run --requirement requirements.txt
+        .venv/bin/pip install --dry-run --requirement requirements.txt
+        ! .venv/bin/pip install --dry-run --requirement requirements.txt 2>/dev/null | grep -q "Would install"