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

Use standard build system #6

Merged
merged 11 commits into from
Feb 3, 2016
Merged

Conversation

WinterMute
Copy link
Contributor

This PR replaces the cmake based build system with a standardised devkitPro Makefile which means that people can build it without installing cmake.

There's still a need to obtain makerom and bannertool but these should not be placed in the $DEVKITARM/bin folder and I'd appreciate it if build instructions didn't encourage people to place anything at all within the toolchain folders. We're currently looking into providing some tools to generate CIAs but that's going to be a little while yet.

@TuxSH
Copy link
Owner

TuxSH commented Feb 2, 2016

Thank you ;)

I'll do two or three tests (should be fine because it's you :p) then merge this PR this WE -- I don't really have time to do dev stuff this week.

@TuxSH
Copy link
Owner

TuxSH commented Feb 2, 2016

Nevermind, had a dozen of minutes to test it today; it's in fact easier than using CMake (here).

Some minor issues:

  • make cia builds the .3dsx version as well, and make clean doesn't delete the generated CIA file.
  • A typo: "TusxSH"
  • Could you set APP_DESCRIPTION to "A tool to backup and restore save files from NDS cartridges (requires either PokeTransporter or Pokemon Dream Radar)" in the PR, please?

@TuxSH
Copy link
Owner

TuxSH commented Feb 3, 2016

Thanks a lot :D.

TuxSH added a commit that referenced this pull request Feb 3, 2016
@TuxSH TuxSH merged commit 0bf5047 into TuxSH:master Feb 3, 2016
@WinterMute WinterMute deleted the standard_build_system branch February 4, 2016 16:22
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