-
Notifications
You must be signed in to change notification settings - Fork 45
Fix Wii and GameCube builds #173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
this got issues again and idk why, and I don't feel like looking at it right now so it'll stay in limbo for a bit lol |
|
I fixed the strange build issues but it still doesn't boot idk why, I'll try to debug this soon |
|
I have no idea why CI is failing, it's not even related to those changes |
Yanis002
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
todo list
| $(BUILD_DIR)/src/overlays/actors/ovl_Item_Shield/%.o: OPTFLAGS := -O2 | ||
| $(BUILD_DIR)/src/overlays/actors/ovl_En_Part/%.o: OPTFLAGS := -O2 | ||
| $(BUILD_DIR)/src/overlays/actors/ovl_Item_B_Heart/%.o: OPTFLAGS := -O0 | ||
| $(BUILD_DIR)/src/overlays/actors/ovl_Bg_Mori_Hineri/%.o: OPTFLAGS := -O0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: test again if this is necessary
|
|
||
| # for ISOs we need to do things manually since we want to remove | ||
| # the useless files that increase the size of the file by a lot | ||
| iso: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: test again gamecube builds
This adds my fixes to make Wii VC and GameCube VC behave properly, the HackerOoT-side should be ready but I still need to complete the basic F3DEX3 support in the emulator, also I need to do a self-review before merging this (and it needs testing)
Also there's audio issues hence the waiting for author label because idk, I'm clueless on this it doesn't do that on vanilla decomp so I'm lost on this one, idk how to debug that (actually I'm building with- update: I fixed it!mips-n64-gcc, it could be the issue)and I swear I'm gonna review the other PRs lol, just haven't found the time for that yet