Skip to content

Fix the Switch CI build against port-maintenance libultraship - #765

Draft
Project516 wants to merge 1 commit into
HarbourMasters:mainfrom
Project516:fix-switch-ci
Draft

Project516 wants to merge 1 commit into
HarbourMasters:mainfrom
Project516:fix-switch-ci

Conversation

@Project516

@Project516 Project516 commented Oct 8, 2026 •

Copy link
Copy Markdown

The Switch build on main fails. CI replaces the pinned libultraship submodule with Net64DD/libultraship main-nx, which is based on libultraship main rather than port-maintenance, so the game can't find APIs it uses:

src/engine/editor/Editor.cpp:55:24: error: 'GetRawInstance' is not a member of 'Ship::Context'

This PR points the Switch job at a branch with the Switch code applied on top of the pinned submodule commit (60c9bb7).

Draft until the libultraship side merges. The workflow currently fetches Project516/libultraship port-maintenance-switch, which is the branch in Net64DD/libultraship#6. Net64DD/libultraship#5 does the same work in a different way. Once one of them is merged, I'll repoint the Switch job back to Net64DD/libultraship and mark this ready.

Testing: the Switch job passes in Release and Debug, built against port-maintenance-switch at ecbdf84 (Net64DD/libultraship#6 after its merge-conflict fix): https://github.com/Project516/SpaghettiKart/actions/runs/39170416022

This branch has not been deployed

No deployments
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.

1 participant