On Tue, 21 Sep 2021, Marcel Taeumel wrote:

> Hi Levente --
> > Yes, but that's not the case with Eliot's changes.
> 
> Loading the new method (override) MethodDictionary >> #atNewIndex:put: first will break the image.

Right, that would indeed break loading.

> 
> Yet, the actual issue was that #at:put: was loaded before #atNewIndex:putValue:. So, I suspect both a bug in the Monticello load order and a more generic problem that might not be so easily fixed.
> 
> The issue with the load order might be easily fixed in MCPackageLoader >> #basicLoad:
> 
> "..."
> "Pass 2: We compile new / changed methods"
> methodAdditions do: "..."

Changes are sorted by MCDependencySorter, but that doesn't do anything 
like what I described. It mixes new methods with updates.
So, what I described is not what is implemented at all. Based on method 
histories, it never was.


Levente