When we load something from an mcz it goes through the MCMethodDefinition>>#asMethodAddition method.
That used to just have a literal 'true' as the parameter for the logSource: part (in 5.3 images) but it got changed a while back to defer to 'SystemChangeNotifier uniqueInstance isBroadcasting'. I think that is incorrect and that it really ought to take into account the acceptsLoggingOfCompilation setting; interestingly, that is how the MCMethodDefinition>>#load handles things.
The immediate issue is wanting to be able to load mczs in a no-source server image, though it looks like there's a bit of work to do to factor out places where code is inadvisably expecting sources to be both available and held in a positionable stream.
Does anyone see a reason to not change MCMethodDefinition>>#asMethodAddition to be a bit more like MCMethodDefinition>>#load in this matter?
tim
--
tim Rowledge; tim@rowledge.org; http://www.rowledge.org/tim
Strange OpCodes: HALT: No-Op