windmill/ansible-role-virtualenv
Code Issues Proposed changes

Fix virtualenv_package_name for Debian

Browse Source 
We actually also need to install the virtualenv package, for the binary.

Change-Id: Ic7a90e27f243a6ec1d94d10c86ef7960323e142d
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2018-12-07 18:34:54 -05:00
parent d5e73c6ba5
commit 9a71107491
4 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yaml
View File

@ -23,8 +23,6 @@ virtualenv_git_version: master
virtualenv_install_method: pip virtualenv_install_method: pip
virtualenv_package_name: python3-virtualenv
virtualenv_pip_name: virtualenv virtualenv_pip_name: virtualenv
#virtualenv_pip_extra_args: #virtualenv_pip_extra_args:
#virtualenv_pip_version: #virtualenv_pip_version:

5
tasks/install/package.yaml
View File

@ -12,6 +12,11 @@
# License for the specific language governing permissions and limitations # License for the specific language governing permissions and limitations
# under the License. # under the License.
--- ---
- name: Define virtualenv_package_name
set_fact:
virtualenv_package_name: "{{ __virtualenv_package_name | list }}"
when: virtualenv_package_name is not defined
- name: Install virtualenv from package - name: Install virtualenv from package
become: yes become: yes
package: package:

4
vars/Debian.yaml
View File

@ -27,3 +27,7 @@
--- ---
__virtualenv_build_depends: __virtualenv_build_depends:
- git - git
__virtualenv_package_name:
- python3-virtualenv
- virtualenv

3
vars/RedHat.yaml
View File

@ -14,3 +14,6 @@
--- ---
__virtualenv_build_depends: __virtualenv_build_depends:
- git - git
__virtualenv_package_name:
- python3-virtualenv