Hi All, but especially release builders,

    there may be an issue with the Flaops registry.  The default
initialization is Flaps class>>#initializeFlapsQuads.  But there are
additional class initializers that add to the Flaps registry, e.g.
TestRunner class>>registerInToolsFlap.  If we unload flaps and then reload
it we will lose all the extra ones, such as TestRunner's.  So for release
we should run all implementors of registerInToolsFlap
or registerInFlapsRegistry (or rather the relevant senders of
registerQuad:forFlapNamed:) to ensure all the classes that want to be in
the Flaps registry are there.

Note that if instead of a Flaps registry there was a class-side "I should
be in the Flaps registry" pragma, then this kind of error wouldn't happen.
Either the pragmas would be collected every time the flaps registry was
opened, or the Flaps registry would be updated whenever a method with the
pragma was added or removed.

On Thu, Jun 1, 2017 at 2:20 PM, <commits at source.squeak.org> wrote:

> Eliot Miranda uploaded a new version of SUnitGUI to project The Trunk:
> http://source.squeak.org/trunk/SUnitGUI-eem.68.mcz
>
> ==================== Summary ====================
>
> Name: SUnitGUI-eem.68
> Author: eem
> Time: 1 June 2017, 2:20:29.979374 pm
> UUID: 47ec06e5-6ff4-4475-9034-b298e6f98544
> Ancestors: SUnitGUI-ul.67
>
> If Flaps is present (and not loading for the first time) run the
> initialize method to make sure TestRunner is in the Flaps Tools registry.
>
> =============== Diff against SUnitGUI-ul.67 ===============
>
> Item was added:
> + (PackageInfo named: 'SUnitGUI') preamble: '"below, add code to be run
> before the loading of this package"
> + "If Flaps is present (and not loading for the first time) run the
> initialize method to make sure TestRunner is in the Flaps Tools registry."
> + (Smalltalk classNamed: #TestRunner) ifNotNil:
> +       [:trc|
> +       trc environment at: #Flaps ifPresent: [ :fc | trc initialize]]'!
>
>
>


-- 
_,,,^..^,,,_
best, Eliot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20170601/8a375c46/attachment.html>