Avoid cyclic imports

Bug #1169594 reported by Marco Giusti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Virtualbricks
Fix Released
Medium
Marco Giusti

Bug Description

How to reproduce the problem:

 $ python -c "from virtualbricks import virtualmachines"

 Traceback (most recent call last):
   File "<string>", line 1, in <module>
   File "/usr/local/lib/python2.7/dist-packages/virtualbricks/virtualmachines.py", line 30, in <module>
  from virtualbricks.bricks import Brick
   File "/usr/local/lib/python2.7/dist-packages/virtualbricks/bricks.py", line 32, in <module>
  from virtualbricks.gui.graphics import Icon, Topology
   File "/usr/local/lib/python2.7/dist-packages/virtualbricks/gui/__init__.py", line 1, in <module>
  from .gui import VBGUI
   File "/usr/local/lib/python2.7/dist-packages/virtualbricks/gui/gui.py", line 34, in <module>
  from virtualbricks.brickfactory import BrickFactory
   File "/usr/local/lib/python2.7/dist-packages/virtualbricks/brickfactory.py", line 40, in <module>
  from virtualbricks.bricks import Brick
 ImportError: cannot import name Brick

This happens when lower modules depends on higer modules and vice versa.
A temporary solution is to import `virtualbricks.gui.gui` (maybe
`virtualbricks.gui.graphics`?) before all the others module but this
patch stinks.

 affects virtualbrick

Changed in virtualbrick:
status: New → Confirmed
importance: Undecided → Medium
Changed in virtualbrick:
status: Confirmed → In Progress
assignee: nobody → mg (marco-giusti)
milestone: none → 1.0
Revision history for this message
Marco Giusti (marco-giusti) wrote :

This is fixed but the commit was lost in time.

Changed in virtualbrick:
status: In Progress → Fix Committed
Changed in virtualbrick:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.