Merge 5d7894d189 into 28cd9be54e

Checks PATH variable for existing inclusion of shims
for shells other than fish.

See PR #430

.github/workflows/tests.yml

@@ -6,9 +6,8 @@ jobs:
       fail-fast: false
       matrix:
         os:
-          - ubuntu-24.04
           - ubuntu-22.04
-          - macos-15
+          - ubuntu-20.04
           - macos-14
           - macos-13
     runs-on: ${{ matrix.os }}

LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2025 Yamashita, Yuu
+Copyright (c) 2015 Yamashita, Yuu
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

README.md

@@ -28,12 +28,6 @@ From inside that directory you can:
  - Get the latest development release by running `git pull` to download the
    latest changes.
 
-💡 **WSL note:** If you're using WSL, we recommend setting Git to use Unix-style line endings to prevent script execution errors:
-
-```sh
-git config --global core.autocrlf input
-```
-
 1. **Check out pyenv-virtualenv into plugin directory**
 
     ```bash
@@ -88,7 +82,7 @@ brew install --HEAD pyenv-virtualenv
 ```
 
 After installation, you'll still need to do
-[Pyenv shell setup steps](https://github.com/pyenv/pyenv#b-set-up-your-shell-environment-for-pyenv)
+[Pyenv shell setup steps](https://github.com/pyenv/pyenv#basic-github-checkout)
 then add 
 ```sh
 eval "$(pyenv virtualenv-init -)"

bin/pyenv-activate

@@ -22,8 +22,10 @@ fi
 
 { printf "\x1B[31;1m"
   echo
-  echo "\`pyenv activate' requires Pyenv and Pyenv-Virtualenv to be loaded into your shell."
+  echo "Failed to activate virtualenv."
-  echo "Check your shell configuration and Pyenv and Pyenv-Virtualenv installation instructions."
+  echo
+  echo "Perhaps pyenv-virtualenv has not been loaded into your shell properly."
+  echo "Please restart current shell and try again."
   echo
   printf "\x1B[0m"
 } 1>&2

bin/pyenv-deactivate

@@ -11,8 +11,10 @@ set -e
 
 { printf "\x1B[31;1m"
   echo
-  echo "\`pyenv deactivate' requires Pyenv and Pyenv-Virtualenv to be loaded into your shell."
+  echo "Failed to deactivate virtualenv."
-  echo "Check your shell configuration and Pyenv and Pyenv-Virtualenv installation instructions."
+  echo
+  echo "Perhaps pyenv-virtualenv has not been loaded into your shell properly."
+  echo "Please restart current shell and try again."
   echo
   printf "\x1B[0m"
 } 1>&2

bin/pyenv-virtualenv

@@ -605,13 +605,7 @@ STATUS=0
 mkdir -p "${PYENV_VIRTUALENV_CACHE_PATH}"
 cd "${PYENV_VIRTUALENV_CACHE_PATH}"
 if [ -n "${USE_CONDA}" ]; then
-  if [ -z "$VIRTUALENV_PYTHON" ]; then
+  pyenv-exec conda create $QUIET $VERBOSE --name "${VIRTUALENV_PATH##*/}" --yes "${VIRTUALENV_OPTIONS[@]}" python || STATUS="$?"
-    #process substitution doesn't seem to work unless it's directly inserted to the command line
-    #if we add it to VIRTUALENV_OPTIONS instead, "broken pipe" happens
-    pyenv-exec conda create $QUIET $VERBOSE --name "${VIRTUALENV_PATH##*/}" --yes "${VIRTUALENV_OPTIONS[@]}" --file <(pyenv-exec conda list python --full-name --export) || STATUS="$?"
-  else
-    pyenv-exec conda create $QUIET $VERBOSE --name "${VIRTUALENV_PATH##*/}" --yes "${VIRTUALENV_OPTIONS[@]}" || STATUS="$?"
-  fi
 else
   if [ -n "${USE_M_VENV}" ]; then
     pyenv-exec "${M_VENV_PYTHON_BIN:-python}" -m venv $QUIET $VERBOSE "${VIRTUALENV_OPTIONS[@]}" "${VIRTUALENV_PATH}" || STATUS="$?"
@@ -631,8 +625,7 @@ fi
 
 ## Create symlink in the `versions` directory for backward compatibility
 if [ -d "${VIRTUALENV_PATH}" ] && [ -n "${COMPAT_VIRTUALENV_PATH}" ]; then
-  # Overwrite an existing link, can happen if running with -f
+  ln -fs "${VIRTUALENV_PATH}" "${COMPAT_VIRTUALENV_PATH}"
-  ln -fsn "${VIRTUALENV_PATH}" "${COMPAT_VIRTUALENV_PATH}"
 fi
 
 if [ ! -e "${VIRTUALENV_PATH}/bin/pydoc" ]; then

bin/pyenv-virtualenv-init

@@ -95,8 +95,10 @@ EOS
   ;;
 * )
   cat <<EOS
+if [[ ":$PATH:" != *"${PYENV_VIRTUALENV_ROOT:-${PYENV_VIRTUALENV_INSTALL_PREFIX}}/shims"* ]]; then
   export PATH="${PYENV_VIRTUALENV_ROOT:-${PYENV_VIRTUALENV_INSTALL_PREFIX}}/shims:\${PATH}";
   export PYENV_VIRTUALENV_INIT=1;
+fi
 EOS
   ;;
 esac

test/conda.bats

@@ -32,8 +32,8 @@ unstub_pyenv() {
   run pyenv-virtualenv venv
 
   assert_success
-  assert_output_wildcards <<OUT
+  assert_output <<OUT
-PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes --file /dev/fd/*
+PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes python
 rehashed
 OUT
 
@@ -56,7 +56,7 @@ OUT
 
   assert_success
   assert_output <<OUT
-PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes python=3.5
+PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes python=3.5 python
 rehashed
 OUT
 
@@ -79,7 +79,7 @@ OUT
 
   assert_success
   assert_output <<OUT
-PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes python=3.5
+PYENV_VERSION=miniconda3-3.16.0 conda create --name venv --yes python=3.5 python
 rehashed
 OUT
 

test/init.bats

@@ -50,8 +50,10 @@ load test_helper
   run pyenv-virtualenv-init - bash
   assert_success
   assert_output <<EOS
+if [[ ":$PATH:" != *"${TMP}/pyenv/plugins/pyenv-virtualenv/shims"* ]]; then
   export PATH="${TMP}/pyenv/plugins/pyenv-virtualenv/shims:\${PATH}";
   export PYENV_VIRTUALENV_INIT=1;
+fi
 _pyenv_virtualenv_hook() {
   local ret=\$?
   if [ -n "\${VIRTUAL_ENV-}" ]; then
@@ -93,8 +95,10 @@ EOS
   run pyenv-virtualenv-init - zsh
   assert_success
   assert_output <<EOS
+if [[ ":$PATH:" != *"${TMP}/pyenv/plugins/pyenv-virtualenv/shims"* ]]; then
   export PATH="${TMP}/pyenv/plugins/pyenv-virtualenv/shims:\${PATH}";
   export PYENV_VIRTUALENV_INIT=1;
+fi
 _pyenv_virtualenv_hook() {
   local ret=\$?
   if [ -n "\${VIRTUAL_ENV-}" ]; then

test/test_helper.bash

@@ -83,16 +83,6 @@ assert_equal() {
   fi
 }
 
-assert_equal_wildcards() {
-  if [[ $1 != $2 ]]; then
-    { echo "expected:"
-      echo "$2"
-      echo "actual:"
-      echo "$1"
-    } | flunk
-  fi
-}
-
 assert_output() {
   local expected
   if [ $# -eq 0 ]; then expected="$(cat -)"
@@ -101,14 +91,6 @@ assert_output() {
   assert_equal "$expected" "$output"
 }
 
-assert_output_wildcards() {
-  local expected
-  if [ $# -eq 0 ]; then expected="$(cat -)"
-  else expected="$1"
-  fi
-  assert_equal_wildcards "$output" "$expected"
-}
-
 assert_output_contains() {
   local expected="$1"
   echo "$output" | grep -F "$expected" >/dev/null || {