search

thefinn93 / ansible-letsencrypt

An ansible role to generate TLS certificates and get them signed by Let's Encrypt
GNU General Public License v3.0
442 stars 122 forks source link

Debian 8 requires virtualenv on top of python-virtualenv #6

Closed gled-rs closed 8 years ago

gled-rs commented 8 years ago

Or pip install fails. python-virtualenv does not install the virtualenv executable.

We can maybe just specify the virtualenv command to use as /bin/true ?

thefinn93 commented 8 years ago

you sure? I've successfully used this role on a number of Debian 8 boxes..

thefinn93 commented 8 years ago

I mean, i guess no reason not to merge. meh.

gled-rs commented 8 years ago

The reason that you had no issue is actually that virtualenv is a Recommand and not a Depend, and thus when we don't install the Recommands by default ( to keep the minimal paquets installed ), it does not install virtualenv.

Thanks for the merge !

thefinn93 commented 8 years ago

Just a heads up, it turns out dropping python-virtualenv package breaks on older versions of Debian or Ubuntu, see #9