On 12 Jun 2013, at 18:07, Jens Timmerman wrote:

> In test/framework/toolchain.py:
> 
> > @@ -225,7 +231,7 @@ def test_optimization_flags_combos(self):
> >  
> >          # check combining of optimization flags (doesn't make much sense)
> >          # lowest optimization should always be picked
> > -        tc = tc_class(version="1.1.0-no-OFED")
> > +        tc = tc_class(version="1.1.0-no-OFED", modules_tool=config.get_modules_tool())
> 
> why is this not the default? since you're adding it everywhere...
> Fixing the circular dependency can fix this.

Yep, that was a nice cleanup triggered by getting rid of the imports, indeed.