[squeak-dev] Re: Uploading mczs to the Trunk (Was: Re: Working
with Trunk and Package loading)
Levente Uzonyi
leves at elte.hu
Mon Mar 8 01:05:11 UTC 2010
On Sun, 7 Mar 2010, Andreas Raab wrote:
> 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.
I like the idea. This gives us another reason to run the tests before
uploading packages.
Levente
>
> Cheers,
> - Andreas
>
>> from PackageDependencyTest>>#testPackage:dependsOnlyOn: seems to be
>> fixing all failures (except #testTools, because Tools also depends on
>> Monticello which it shouldn't).
>
>
More information about the Squeak-dev
mailing list
|