Merge pull request #16 from brabster/pip_require_virtualenv

check PIP_REQUIRE_VIRTUALENV
brabster · Oct 1, 2024 · 1f854b9 · 1f854b9
2 parents 0495dea + 3e4b952
commit 1f854b9
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/.github/workflows/test_install.yml b/.github/workflows/test_install.yml
@@ -38,9 +38,9 @@ jobs:
       - name: check
         shell: bash -l {0}
         run: |
-          su - tester
-          alias
-          git config --list
-          python --version
-          clamscan --version
+          su - tester -c "alias"
+          su - tester -c "git config --list"
+          su - tester -c "python --version"
+          su - tester -c "! pip install jsonschema # should not work because no venv"
+          su - tester -c "clamscan --version"
           
diff --git a/roles/pyenv/tasks/main.yml b/roles/pyenv/tasks/main.yml
@@ -27,6 +27,7 @@
     path: '{{ user_profile }}'
     marker: '# {mark} ANSIBLE_PYENV'
     block: |
+      export PIP_REQUIRE_VIRTUALENV=true
       export PATH="{{ user_home }}/.pyenv/shims:{{ user_home }}/.pyenv/bin:$PATH"
       eval "$(pyenv init -)"
       eval "$(pyenv virtualenv-init -)"

diff --git a/workstation.yml b/workstation.yml
@@ -16,8 +16,6 @@
   - role: ./roles/gcloud
   - role: ./roles/expressvpn
   - role: ./roles/pyenv
-  - role: ./roles/pipx
-  - role: ./roles/pipenv
   - role: ./roles/git
   - role: ./roles/clamav
   - role: ./roles/xfce