Skip to content

ido7.1_irix4 added as script in ido7.1 - #79

Open
jtl3d wants to merge 4 commits into
decompme:mainfrom
jtl3d:ido7.1-irix4-script
Open

ido7.1_irix4 added as script in ido7.1#79
jtl3d wants to merge 4 commits into
decompme:mainfrom
jtl3d:ido7.1-irix4-script

Conversation

@jtl3d

@jtl3d jtl3d commented Aug 8, 2026

Copy link
Copy Markdown

Companion PR for decompme/decomp.me#2068

Comment thread platforms/n64/ido7.1/cc-irix4 Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. This should be an external dependency like other examples

@mkst

mkst commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

OK, sat at a laptop rather than on my phone.

What you need to do is host your script somewhere (github repo or a gist?) and then add the url (commit hash rather than just "main" so we can have some level of protection against anyone replacing that script with something nasty).

And then simply add it to the list of files that are downloaded, e.g. see
https://github.com/decompme/compilers/blob/main/values.yaml#L73C2-L81C97

.. with that said.. you also need to host the ido binary along with qemu-irix and whatever else you need to run the compiler.

I'd try to take a pause to understand the requirements, rather than just asking an LLM to do everything (which is what this feels like - please correct me if im wrong).

@jtl3d

jtl3d commented Aug 9, 2026

Copy link
Copy Markdown
Author

Can I not piggy back off the other compilers? The whole point of the script is it uses existing compilers already defined (IDO4 has qemu-irix bundled already). So if I just hosted the script in a gist, would that be enough? Or does it have to be self-contained?

@mkst

mkst commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

I don't think the other IDO compilers use qemu, I strongly suspect they are the statically recompiled binaries.

@jtl3d

jtl3d commented Aug 9, 2026

Copy link
Copy Markdown
Author

I updated to pull from a gist. ido4.1 has qemu-irix at usr/bin/qemu-irix-4.0, so that's why this works. ido7.1 being recompiled is fine for this setup. Please let me know if anything else looks wrong!

@mkst

mkst commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

ok, this looks in much better shape. Can you confirm you've tested this script locally and it works as you expect?

@jtl3d

jtl3d commented Aug 9, 2026

Copy link
Copy Markdown
Author

Yes it's working locally for me:
image

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