[WIP] Fix for copying Py2 deps. Unlike py3, tox creates symlinks for py2 #9
Conversation
JamieCressey
changed the title
| @@ -31,7 +31,13 @@ def install_debian_deps(path, deps, opts, action): | |||
| tmp_usr = path_join(tmp_dir, 'usr') | |||
| if isdir(tmp_usr): | |||
| action.setactivity('copy', ', '.join(__iter_files(tmp_usr))) | |||
| action.popen(['cp', '-r'] + listdir(tmp_usr) + [path], cwd=tmp_usr) | |||
| _dirs = listdir(tmp_usr) | |||
| if 'py2' in path: | |||
There was a problem hiding this comment.
I might be wrong here, but I'm wondering if this reliable enough. Maybe we could use action.venv to find out if it's a Python 2 environment or ask the Python interpreter inside the env for its version or something. What do you think?
There was a problem hiding this comment.
Agreed, it's not ideal. I'll see what action.venv provides.
On 17 Jan 2016 11:45, "Attila Magyar" [email protected] wrote:
In tox_DEBIAN/install.py
#9 (comment)
:@@ -31,7 +31,13 @@ def install_debian_deps(path, deps, opts, action):
tmp_usr = path_join(tmp_dir, 'usr')
if isdir(tmp_usr):
action.setactivity('copy', ', '.join(__iter_files(tmp_usr)))
action.popen(['cp', '-r'] + listdir(tmp_usr) + [path], cwd=tmp_usr)_dirs = listdir(tmp_usr)if 'py2' in path:I might be wrong here, but I'm wondering if this reliable enough. Maybe we
could use action.venv to find out if it's a Python 2 environment or ask
the Python interpreter inside the env for its version or something. What do
you think?—
Reply to this email directly or view it on GitHub
https://github.com/balabit/tox-debian-plugin/pull/9/files#r49945046.
No description provided.