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

Modernize MakeROM build system + bug fixes #120

Merged
merged 40 commits into from
Apr 17, 2022
Merged

Modernize MakeROM build system + bug fixes #120

merged 40 commits into from
Apr 17, 2022

Conversation

jakcron
Copy link
Collaborator

@jakcron jakcron commented Apr 17, 2022

About

The rewrite for MakeROM isn't complete yet, but to pave the way for that, MakeROM's build system needs updating. Also migrating from polarssl to mbedtls allowed me to implement proper CCI initial data generation (#107).

Structural Changes

  • MakeROM uses a makefile similar to CTRTool.
  • MakeROM's directory structure matches CTRTool's directory structure.
  • Support for building with CYGWIN using makefile has been removed.
  • MakeROM's Visual Studio project files have been removed.
  • polarssl was replaced by mbedtls.
  • MakeROM's 3rd party code libblz libyaml & libmbedtls are now broken off into separate libraries that are built and included as static libraries.
  • Added documentation for usage and compiling source.
  • Root README is more rich.

Functional Changes

  • Bumped version to v0.18.1.
  • [BugFix] Fixes bug where initial data wasn't generated correctly ( [Bug] MakeROM does not create valid CardInfo/DevCardInfo region. #107 ).
  • [Change] Failures in generating RSA signatures is now a warning, not an error.
  • [Change] MakeROM will now give more information about fatal errors. This should help diagnose confusing errors.
  • [Change] MakeROM will warn users if the CCI crypto-type isn't supported by the target.

@jakcron jakcron merged commit cc707c1 into master Apr 17, 2022
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