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

Remove the ROM and component binaries from the disassembly #1

Open
halian opened this issue Jan 17, 2022 · 3 comments
Open

Remove the ROM and component binaries from the disassembly #1

halian opened this issue Jan 17, 2022 · 3 comments

Comments

@halian
Copy link

halian commented Jan 17, 2022

…before Nintendō catches wind and issues a takedown notice. Please. 😟

@brianhuffman
Copy link
Owner

That's probably a good idea. I suppose I can use a checksum or a hash of the code section for regression testing instead of comparing with a known-good binary. And for building the full cartridge image, I can host the tileset binary privately instead of storing it publicly on github. Maybe I could add a script that can extract the tileset from a standard rom image that users could obtain from anywhere else.

If I want to be paranoid, it might be a good idea to rewrite the git history to remove all traces of those binaries from all the previous commits as well. What do you think?

Anway, thanks for the suggestion! It's nice to see that someone has noticed this little project :)

@halian
Copy link
Author

halian commented Jan 18, 2022

Rewriting the git history would be a good idea. Most cartridge decomps that can't build ab nihilo require the user to provide their own ROM, and give checksums against which to confirm the build.

@brianhuffman
Copy link
Owner

I did some git filter-branch surgery to remove some files from the history, and force-pushed. Before closing this ticket, I'd like to take care of two additional things:

  • Update the build/test script to compute and compare hashes
  • Add a script to extract the tileset from a full rom image

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

No branches or pull requests

2 participants