Code review comment for lp:~savoirfairelinux-openerp/lp-community-utils/pep394

Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

Hello Sandy,

Ik rechecked the effect of the different commands on our virtualenv installation running on debian testing.

As can be seen /usr/bin/env python2 is using the system python, even when running from a virtualenv. I think you are using a much newer version of virtualenv, that is creating the symbolic links you are referring to. My point is that a lot of existing - even relatively recent and up to date - installations, do not have those symlinks.

rportier@vanaheim:~$ virtualenv --python=python2.7 test_usr_bin_env
Running virtualenv with interpreter /usr/bin/python2.7
New python executable in test_usr_bin_env/bin/python2.7
Also creating executable in test_usr_bin_env/bin/python
Installing distribute.............................................................................................................................................................................................done.
Installing pip...............done.
rportier@vanaheim:~$ ls -l
total 24
-rw------- 1 rportier people 1643 Jan 3 15:59 <email address hidden>
-rw------- 1 rportier people 1070 Jan 3 15:59 <email address hidden>
-rw------- 1 rportier people 1751 Jan 3 15:57 <email address hidden>
-rw------- 1 rportier people 4448 Jan 3 16:00 <email address hidden>
drwxr-xr-x 6 rportier people 4096 Feb 7 10:46 test_usr_bin_env
rportier@vanaheim:~$ . test_usr_bin_env/bin/activate
(test_usr_bin_env)rportier@vanaheim:~$ python
Python 2.7.3 (default, Jan 2 2013, 13:56:14)
[GCC 4.7.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.prefix
'/home/rportier/test_usr_bin_env'
>>> quit()
(test_usr_bin_env)rportier@vanaheim:~$ /usr/bin/env python
Python 2.7.3 (default, Jan 2 2013, 13:56:14)
[GCC 4.7.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.prefix
'/home/rportier/test_usr_bin_env'
>>> quit()
(test_usr_bin_env)rportier@vanaheim:~$ /usr/bin/env python2
Python 2.7.3 (default, Jan 2 2013, 13:56:14)
[GCC 4.7.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.prefix
'/usr'

« Back to merge proposal