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

mame.lst: update primrag2 string #13339

Closed
wants to merge 1 commit into from

Conversation

mikehyrus
Copy link
Contributor

Forgot this one by accident.

Forgot this one by accident.
@happppp
Copy link
Member

happppp commented Feb 7, 2025

To be honest, I don't care about all the comments in this file, I'd rather they be removed. It's a setname index for the compile script, not something users are meant to browse/read.

@mamehaze
Copy link
Contributor

mamehaze commented Feb 7, 2025

ideally the file needs to go away at some point too, but it seems there are still issues with making that step

@cuavas
Copy link
Member

cuavas commented Feb 7, 2025

To be honest, I don't care about all the comments in this file, I'd rather they be removed. It's a setname index for the compile script, not something users are meant to browse/read.

Want me to do that with a script?

@happppp
Copy link
Member

happppp commented Feb 7, 2025

I already have a local commit removing them for tiny.lst, give me a second, I'll push that one first.
And yeah, if noone minds it, I think the ones in mame.lst can be removed.

@happppp happppp closed this Feb 7, 2025
@mikehyrus mikehyrus deleted the roman_numerage2 branch February 8, 2025 03: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.

4 participants