opensmalltalk-vm icon indicating copy to clipboard operation
opensmalltalk-vm copied to clipboard

Repair linux64x64 compilation

Open tonyg opened this issue 1 year ago • 3 comments

Hello Eliot et al.,

I produced these commits to get Cog building on Debian sid/trixie from current git Cog branch. The number of faults suggested to me that I was holding it wrong but I couldn't figure out what specifically I was doing; if I have been an idiot, and these commits are unnecessary, please let me know because I'll be happy not to have needed them :-)

The commits can be applied individually. I wrote them in a hurry; hopefully they're reasonable?

There's one, de1cb25 (the "workaround" one) that should probably not be applied, but that I include for completeness because otherwise I do not seem to be able to get the build to go through.

Cheers, Tony

tonyg avatar Jul 08 '24 08:07 tonyg

I wonder in the first place what the code cleanup was/or is in OpenSmalltalk that has happened over the last few weeks. Also see the report in https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/685 .

cstes avatar Aug 18 '24 17:08 cstes

Hi Tony,

apologies for not having attended to these sooner. I'll try and get to them this week. I'm hopefully nearing the end of a Slang transpiler rewrite that is consuming all my cycles and head space ATM.

Cheers.

On Mon, Jul 8, 2024 at 1:12 AM Tony Garnock-Jones @.***> wrote:

Hello Eliot et al.,

I produced these commits to get Cog building on Debian sid/trixie from current git Cog branch. The number of faults suggested to me that I was holding it wrong but I couldn't figure out what specifically I was doing; if I have been an idiot, and these commits are unnecessary, please let me know because I'll be happy not to have needed them :-)

The commits can be applied individually. I wrote them in a hurry; hopefully they're reasonable?

There's one, de1cb25 https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/de1cb25f24bfa10f22819f9f287c96e37557746d (the "workaround" one) that should probably not be applied, but that I include for completeness because otherwise I do not seem to be able to get the build to go through.

Cheers, Tony

You can view, comment on, or merge this pull request online at:

https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/688 Commit Summary

File Changes

(9 files https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/688/files)

Patch Links:

  • https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/688.patch
  • https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/688.diff

— Reply to this email directly, view it on GitHub https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/688, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADY5VUEI7HSX7PVGOTWTXCDZLJCXVAVCNFSM6AAAAABKQJSD3GVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM4TIOJVGQZTONY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- ,,,^..^,,, best, Eliot

eliotmiranda avatar Aug 20 '24 18:08 eliotmiranda

On 8/20/24 20:56, Eliot Miranda wrote:

apologies for not having attended to these sooner. I'll try and get to them this week. I'm hopefully nearing the end of a Slang transpiler rewrite that is consuming all my cycles and head space ATM.

No problem, I'm immensely grateful for your work. Exciting to hear about the rewrite. Best of luck!

tonyg avatar Aug 21 '24 16:08 tonyg

As the Cog branch builds and runs OK for me on linux64x64 now without this PR's changes, I think it's not required anymore. Hooray!

tonyg avatar Aug 27 '25 12:08 tonyg