Skip to content

Resolve a cyclic dependency#4

Merged
stankevich merged 1 commit intovoxpupuli:masterfrom
zoni:cyclicdependency
Mar 14, 2013
Merged

Resolve a cyclic dependency#4
stankevich merged 1 commit intovoxpupuli:masterfrom
zoni:cyclicdependency

Conversation

@zoni
Copy link
Copy Markdown
Contributor

@zoni zoni commented Mar 11, 2013

There was a dependency between python::virtualenv and
python::requirements. Basically, the virtualenv wanted to initialize
requirements, but requirements depended on the virtualenv being created
already, resulting in:

err: Could not apply complete catalog: Found 1 dependency cycle:
(Exec[python_requirements_check_/usr/share/err/repo/requirements.txt] =>
Python::Requirements[/usr/share/err/repo/requirements.txt] =>
Exec[python_virtualenv_/usr/share/err/python3] =>
Python::Requirements[/usr/share/err/repo/requirements.txt] =>
Exec[python_requirements_check_/usr/share/err/repo/requirements.txt])

This resolves that circular dependency and, as far as I can tell, does not negatively affect anything else.

There was a dependency between python::virtualenv and
python::requirements. Basically, the virtualenv wanted to initialize
requirements, but requirements depended on the virtualenv being created
already, resulting in:

err: Could not apply complete catalog: Found 1 dependency cycle:
(Exec[python_requirements_check_/usr/share/err/repo/requirements.txt] =>
Python::Requirements[/usr/share/err/repo/requirements.txt] =>
Exec[python_virtualenv_/usr/share/err/python3] =>
Python::Requirements[/usr/share/err/repo/requirements.txt] =>
Exec[python_requirements_check_/usr/share/err/repo/requirements.txt])
@stankevich stankevich merged commit 451b215 into voxpupuli:master Mar 14, 2013
@zoni zoni deleted the cyclicdependency branch March 14, 2013 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants