Seems it's broken

[pytest-dev]

Seems it's broken

[nicoddemus]

Oh, many thanks for reporting this. It was working without a glitch for a while, so it would take me a while to realize it was not working correctly. 😄

[nicoddemus]

Hey,

I did take a look at the problem, but it seems the solution is more complicated than I thought, seems to be a problem on how tox handles the --force-dep option. This might take awhile.

Thanks again for the report! 😄

[ionelmc]

Can you elaborate on what's the problem? --force-dep doesn't seem related to that weird error about requirements.txt

[nicoddemus]

Sure!

From the stack trace:

File "/home/travis/virtualenv/python2.7.9/src/tox/tox/_config.py", line 374, in _makeenvconfig
    name = self._replace_forced_dep(name, config)
  File "/home/travis/virtualenv/python2.7.9/src/tox/tox/_config.py", line 432, in _replace_forced_dep
    if self._is_same_dep(forced_dep, name):
  File "/home/travis/virtualenv/python2.7.9/src/tox/tox/_config.py", line 443, in _is_same_dep
    dep2_name = pkg_resources.Requirement.parse(dep2).project_name

The problem lies in constructing the pkg_resources.Requirement.parse object, which is used when the --force-dep option is passed. This happens when a plugin lists its dependencies using a requirements.txt file like this:

[testenv]
deps = -r{toxinidir}/requirements-testing.txt

Instead of defining them explicitly:

[testenv]
deps = foo==1.0
   bar>1.2

In this case "-r /path/to/requirements.txt" is being passed instead of a project version requirement to pkg_resources.Requirement.parse. I didn't looked in the problem too deeply, but it seems like tox in the normal workflow just forwards deps to pip install machinery, but when the --force-dep option is passed it actually tries to parse dependencies to find the one to substitute versions and breaks. And that is my fault as I wrote support for --force-dep. 😄

The correct solution would be to parse the requirements (using pkg_resources), but hadn't have time to look at it yet as I was on vacation. But I will try to get to this next week.

Cheers,