I am 100% confident that we can fix this easily (i.e. portability across 7, 8 or 9), there is not a lot of code and it looks pretty straightforward at first sight.
> On 21 May 2021, at 18:23, ducasse <[email protected]> wrote: > > Thanks. > I do not think that it is good to reintroduce this super ugly class. at the > minimum it should be in a separate branch. > > S > >> On 21 May 2021, at 18:06, Günter Khyo via Pharo-dev >> <[email protected]> wrote: >> >> Hi Stef, >> >> Great! I'll send you a pull-request for the "hotfix" this evening. As I >> mentioned to Sven, I'll do some rework on the implementation and >> documentation when I find the time. >> >> Günter >> >> On 5/21/21 5:55 PM, ducasse wrote: >>> I checked and all the tests pass on Pharo80. >>> Now I did a little pass on the tests. >>> I do not have the time to migrate this lib to P9. >>> But I would accept Pull Requests :) >>> >>> S. >>> >>>> On 21 May 2021, at 17:46, ducasse <[email protected]> wrote: >>>> >>>> The old travis was telling that the code works in 6.1, 7 and 8.0 >>>> I added GithubActions for Pharo 80 and I will see. >>>> >>>> S >>>> >>>>> On 21 May 2021, at 17:40, ducasse <[email protected]> wrote: >>>>> >>>>> Hi gunter >>>>> >>>>> I’m maintaining OSC >>>>> https://github.com/Ducasse/OSC >>>>> and TUIO >>>>> https://github.com/Ducasse/TUIO >>>>> >>>>> Please do some PR if something does not work. >>>>> >>>>> S >>>>> >>>>>> On 21 May 2021, at 14:02, Günter Khyo via Pharo-dev >>>>>> <[email protected]> wrote: >>>>>> >>>>>> Hi, >>>>>> >>>>>> I have ported the Open Sound Control package to Pharo 9.0. Since the >>>>>> Catalog Browser is marked as legacy, I was wondering how to submit the >>>>>> patch. Is there a git repository? I attached the fileout to this email >>>>>> in case anybody wants to take a look at the package or knows how/where >>>>>> to add it to Pharo 9.0, >>>>>> >>>>>> Patch Note: >>>>>> >>>>>> OSC relies on the removed RWTextOrBinaryStream class which has very >>>>>> specific behavior that I could not emulate using the available stream >>>>>> and codec classes, so I pulled in the class from Squeak 5, renamed it to >>>>>> OSCStream and added it to the OSC package. This seemed to be the easiest >>>>>> way to do it without changing the OSC implementation, but I welcome any >>>>>> suggestions for a cleaner solution. >>>>>> >>>>>> Günter >>>>>> >>>>>> <OSC.st> >>>>> >>>> >>> >
