[DOCS-25] if requirements.txt changes, you must delete your .tox folder Created: 06/Jul/16 Updated: 02/Apr/17 Resolved: 02/Apr/17 |
|
| Status: | Resolved |
| Project: | docs |
| Component/s: | Toolchain |
| Affects Version/s: | unspecified |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Colin Dixon | Assignee: | Colin Dixon |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 6164 |
| Description |
|
When locally testing the patch [0] which added sphinx_bootstrap_theme to the requirements.txt, I got the following error: > spectre:docs ckd$ tox -edocs Deleting the .tox directory and running tox -edocs again fixed it, but this seems like a weird failure case that tox should catch. [0] https://git.opendaylight.org/gerrit/#/c/41370/ |
| Comments |
| Comment by Thanh Ha (zxiiro) [ 06/Jul/16 ] |
|
Looks like tox has this tracked as a bug: https://bitbucket.org/hpk42/tox/issues/149/virtualenv-is-not-recreated-when-deps There's workarounds but I don't think any of them are pretty as they require you to script around the problem further complicating the build. I think we should at least document that you may need to clear your .tox directory if requirements change. |
| Comment by Colin Dixon [ 06/Jul/16 ] |
|
I'll fold that into my patch here: |
| Comment by Thanh Ha (zxiiro) [ 02/Apr/17 ] |
|
I think we're done here with the documentation update. Colin please reopen if you disagree. |