[Cuis-dev] Using OSProcess package

[Juan Vuletich]

On 12/10/2022 2:00 PM, David T. Lewis via Cuis-dev wrote:
> Hi Juan,
>
> On Fri, Dec 09, 2022 at 03:37:25PM -0300, Juan Vuletich wrote:
>> Hi Dave,
>>
>> On 12/9/2022 2:33 PM, David T. Lewis via Cuis-dev wrote:
>>> Hi Juan,
>>>
>>> I have not had time to look in detail, but something is badly broken. I see
>>> "Bad file descriptor" errors in the console, and
>>> UnixProcessAccessorTestCase
>>> crashes the VM.
>>>
>>> I'll try to look at it this weekend and give a better answer.
>>>
>>> Dave
>> The problem is that what used to be instances of String, that can be
>> sent directly to the C world, only needing a zero byte at the end, are
>> now instances of UnicodeString. UnicodeString is a regular class (with
>> instance variables) and it can't be sent to the C world in the same way.
>> Something like UnicodeString that uses encapsulation to protect its
>> encoding is much better for the Smalltalk world. But unfortunately it
>> breaks the assumptions of code that deals with the external world.
>> Apologies for that.
>>
> Would it be all right to add the attached UniFileStream>>fileID to
> Cuis? It's needed for OSProcess and it would help if we can have
> it in the base image.
>
> I made the changes and pushed the OSProcess update to github. Thanks
> for the guidance and explanation. I used the #asByteString approach
> because it was simple and easy to keep compatible with Squeak.
>
> OSProcess is working again now, but requires the UniFileStream>>fileID
> update before the tests can be run. I note that there problems in the
> AioEventHandlerTestCase, but I think this is a separate issue (maybe
> a difference in how Semaphore works on Squeak and Cuis). I don't
> think that this will cause problems for most OSProcess users.
>
> Dave
>

Thanks Dave for looking into this. I just integrated the #fileID accessor.

If there's anything I can help with, just let me know.

Cheers,

-- 
Juan Vuletich
cuis.st
github.com/jvuletich
researchgate.net/profile/Juan-Vuletich
independent.academia.edu/JuanVuletich
patents.justia.com/inventor/juan-manuel-vuletich
linkedin.com/in/juan-vuletich-75611b3
twitter.com/JuanVuletich