brianhuffman / drmario

Disassembly of Dr. Mario NES game.
8 stars 1 forks source link

Remove the ROM and component binaries from the disassembly #1

Open halian opened 2 years ago

halian commented 2 years ago

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

brianhuffman commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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: