<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"><html><head><meta content="text/html;charset=UTF-8" http-equiv="Content-Type"></head><body ><div style="font-family: Verdana, Arial, Helvetica, sans-serif; font-size: 10pt;"><div>Hi Tom , Jakob<br></div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><div><div><br></div><div style="" data-zbluepencil-ignore="true" class="zmail_extra"><blockquote style="border-left: 1px solid rgb(204, 204, 204); padding-left: 6px; margin: 0px 0px 0px 5px;"><div><div dir="ltr"><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 :)):<br></div><div><br></div><div><div>SubscriptionRegistry>>getInteractionsForClass: eventClass<br></div><div>"Return the list of subscription for a given Event class"<br></div><div>| answer |<br></div><div>answer := OrderedCollection new.<br></div><div>subscriptionsByAnnouncementClasses values do: [ :collection | <br></div><div>collection do: [:subscription |<br></div><div>(subscription action receiver class includesBehavior: eventClass) ifTrue: [answer add: subscription subscriber]]].<br></div><div>^ answer<br></div></div><div><br></div><div><div>SubscriptionRegistry>>handleSubscriberClass: eventClass<br></div><div>"Return true if the receiver has a callback subscripbed for the event class"<br></div><div>^ subscriptionsByAnnouncementClasses values anySatisfy: [ :subCollection | <br></div><div>subCollection anySatisfy: [:subscriber | subscriber action receiver class includesBehavior: eventClass ]]<br></div></div><div><br></div><div>With these two, the two RSRoassal3Test methods pass for me.<br></div></div></div></div></blockquote></div><div><br></div></div></blockquote><div><br></div><div>I will use yours as I have no idea how to "make better use of our stlib" (:<br></div><div>I did some quick checking and found, as you did, that Pharo has rolled their own: <a href="https://github.com/pharo-project/pharo-core/tree/6.0/Announcements-Core.package" target="_blank">https://github.com/pharo-project/pharo-core/tree/6.0/Announcements-Core.package</a><br></div><div><br></div><div style="" data-zbluepencil-ignore="true" class="zmail_extra"><div><br></div><div><br></div></div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><div><div>Is AXAnnouncements still API-compatible with the Pharo Announcements (or<br></div><div>should I phrase this the other way around)? Either way, it would be nice to<br></div><div>have a working Pharo-Announcements-API implementation for Squeak, at least<br></div><div>for compatibility's sake. It doesn't have to be in the Trunk, but once you<br></div><div>have an implementation or shim, one would extend the BaselineOfRoassal to<br></div><div>include this dependency for Squeak only.<br></div><div class="zmail_extra" data-zbluepencil-ignore="true" style=""><div><br></div></div></div></blockquote><div style="" data-zbluepencil-ignore="true" class="zmail_extra"><div id="Zm-_Id_-Sgn1"><br></div><div id="Zm-_Id_-Sgn1"><br></div><div id="Zm-_Id_-Sgn1">I remember reading years ago that Announcements and AXAnnouncements where API compatible, I have no idea on Pharo.</div><div id="Zm-_Id_-Sgn1"><br></div><div id="Zm-_Id_-Sgn1">So, to compare the API, just make sure each implements the messages and returns the same stuff? Check that tests implemented in AXA work in Pharo and tests in Pharo Announcments work in AXA Squeak?<br></div><div id="Zm-_Id_-Sgn1"><br></div><div><div><br></div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><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.<br></div></blockquote><div><br></div><div class="zmail_extra" data-zbluepencil-ignore="true" style=""><div>I will do that this weekend! <br></div><div><br></div><div>cheers,<br></div><div><br></div><div>t</div></div><div><br></div></div></div><div><br></div></div><br></body></html>