<div dir="ltr"><div>Hi,</div><div><br></div><div>@timothy, happy to hear that Roassal now loads for you! :)<br></div><div><br></div><div>Concerning Announcements: when I initially tried to load Roassal, I included AXAnnouncements from Squeaksource. However, there are some extension methods from Roassal on Announcements that assume a different data layout. In the version on Github I just made sure the extensions compiled at all. Looking at the failing tests now, here are the two hopefully correct implementations for the extension methods (code is mostly copied from the Pharo version, you may want to consider having these make better use of our stdlib if we want to keep those :)):</div><div><br></div><div>SubscriptionRegistry>>getInteractionsForClass: eventClass<br>        "Return the list of subscription for a given Event class"<br>   | answer |<br>    answer := OrderedCollection new.<br>      subscriptionsByAnnouncementClasses values do: [ :collection | <br>                collection do: [:subscription |<br>                       (subscription action receiver class includesBehavior: eventClass) ifTrue: [answer add: subscription subscriber]]].<br>    ^ answer</div><div><br></div><div>SubscriptionRegistry>>handleSubscriberClass: eventClass<br> "Return true if the receiver has a callback subscripbed for the event class"<br>        ^ subscriptionsByAnnouncementClasses values anySatisfy: [ :subCollection | <br>           subCollection anySatisfy: [:subscriber | subscriber action receiver class includesBehavior: eventClass ]]</div><div><br></div><div>With these two, the two RSRoassal3Test methods pass for me.</div><div><br></div><div>I think the most helpful next step would be to go through each package, make sure the package can be loaded without stumbling over stray pharo-only symbols and getting all the tests to pass.</div><div><br></div><div>Best,</div><div>Tom<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 11, 2020 at 9:58 PM Jakob Reschke <<a href="mailto:forums.jakob@resfarm.de">forums.jakob@resfarm.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">"We" might not need Announcements, but Roassal uses them, so "it" needs them.<br>
:-)<br>
<br>
Is AXAnnouncements still API-compatible with the Pharo Announcements (or<br>
should I phrase this the other way around)? Either way, it would be nice to<br>
have a working Pharo-Announcements-API implementation for Squeak, at least<br>
for compatibility's sake. It doesn't have to be in the Trunk, but once you<br>
have an implementation or shim, one would extend the BaselineOfRoassal to<br>
include this dependency for Squeak only.<br>
<br>
<br>
<br>
--<br>
Sent from: <a href="http://forum.world.st/Squeak-Dev-f45488.html" rel="noreferrer" target="_blank">http://forum.world.st/Squeak-Dev-f45488.html</a><br>
<br>
</blockquote></div>