Skip to content

Conversation

@sozud
Copy link
Contributor

@sozud sozud commented Oct 7, 2025

Relies on decompme/compilers#44
Seems to work fine locally by manually copying the compilers into backend/compilers/dreamcast

@sozud
Copy link
Contributor Author

sozud commented Oct 14, 2025

I updated this to include the additional compilers from here decompme/compilers#46

@sozud
Copy link
Contributor Author

sozud commented Oct 20, 2025

I see the CI was kicked off again for this, are the failing tests what is blocking this from being merged? From what I see a bunch of unrelated compilers are failing and it's the same for every PR

@mkst
Copy link
Collaborator

mkst commented Oct 20, 2025

Yeh I think it's an issue with dosemu2.. but there's no point merging because we can't release else we'll hit the same issue in production.

I was going to try and create a simple reproduction so I can raise the issue with the dosemu2 (beyond "you've broken our stuff").. I just haven't found the energy to do it yet 😞

@sozud
Copy link
Contributor Author

sozud commented Oct 20, 2025

We've run into dosemu breakage with the PPA previously so we switched to having our own debs. Here's the Dockerfile https://github.com/sozud/dosemu-deb/blob/main/Dockerfile

@mkst
Copy link
Collaborator

mkst commented Oct 21, 2025

I've managed to repro locally and get the error message, so have created an issue with the dosemu2 folks, they're usually incredibly fast to resolve issues whenever I've raised them in the past.

@mkst
Copy link
Collaborator

mkst commented Oct 21, 2025

Sorry, I expected it to pass now that I've fixed up dosemu2. I'll merge this now, thanks dude!

@mkst mkst merged commit 1ba4314 into decompme:main Oct 21, 2025
17 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants