Skip to content
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

Xbox next #760

Closed
wants to merge 4 commits into from
Closed

Xbox next #760

wants to merge 4 commits into from

Conversation

AJenbo
Copy link
Member

@AJenbo AJenbo commented Jul 19, 2020

This is the next step for mainlining the Xbox port, I have cleaned up the commmit history, but it still needs to be tested by @Marty28 before we can safly replace the old PR.

@AJenbo AJenbo mentioned this pull request Jul 19, 2020
12 tasks
@AJenbo AJenbo changed the base branch from master to ui-refactor July 19, 2020 18:46
Brentdcnz and others added 3 commits July 19, 2020 20:59
This will help with porting the game to the original Xbox, and also make
it easier to implement dynamic resolutions and translations.
@AJenbo AJenbo force-pushed the ui-refactor branch 5 times, most recently from 9c20356 to 6c2dea8 Compare July 22, 2020 10:48
Base automatically changed from ui-refactor to master September 9, 2020 03:14
@glebm
Copy link
Collaborator

glebm commented Jul 1, 2022

This port may be possible with nxdk now, because:

  1. nxdk does not have SDL_Mixer => We no longer use SDL_Mixer.
  2. nxdk does not support exceptions nor RTTI => We can now build without exceptions and RTTI (set(NONET ON)).
  3. nxdk supports modern C++, no need for C++03

We probably don't need any of the patches here anymore, it should be able to build the nxdk version like we do any other port.

Tried to build for NXDK in #4838 but there are still a few build errors.

@AJenbo
Copy link
Member Author

AJenbo commented Jul 1, 2022

Since marty28 appears to have abandoned GitHub I think we can just close this at this point as it was mostly untested, reported to possibly have an issue, and heavily outdated at this point, no CI.

@AJenbo AJenbo closed this Jul 1, 2022
@AJenbo AJenbo deleted the xbox-next branch July 1, 2022 16:02
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.

3 participants