On 3/7/2010 3:18 PM, Levente Uzonyi wrote:
> I think there's an unnecessary assert in
> PackageDependencyTest>>#testPackage:dependsOnlyOn: which causes failure
> if the package doesn't depend on a package which is contained by pkgList.
> For example Compiler doesn't depend on Tools (according to
> DependecyBrowser), but PackageDependencyTest>>#testCompiler passes Tools
> as a possible dependency.
> Removing this line
> self assert: (pkgDeps includesKey: pkg).

This choice was originally deliberate, since I'd like us to keep track 
of 'em explicitly, so if we remove a dependency we should update the 
test and try hard not to reintroduce it. However, since this test is 
supposed to help us in our work, we might reconsider the choice, if 
people feel that's not appropriate.

Cheers,
   - Andreas

> from PackageDependencyTest>>#testPackage:dependsOnlyOn: seems to be
> fixing all failures (except #testTools, because Tools also depends on
> Monticello which it shouldn't).