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

Better Firmware Names #3766

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Better Firmware Names #3766

wants to merge 2 commits into from

Conversation

hegyak
Copy link
Contributor

@hegyak hegyak commented Sep 7, 2023

Renaming Firmware Files based on System Name as BizHawk uses or at least renamed to be related to what it's for.

This changes the Auto-Renamed Firmware Files to be named based on the System ID.
Or to be more Descriptive. So the users can see helpful file names.

The File has been formatted to use LF instead of CRLF. Which is why it feels like a "Full Replacement"

Examples of Old Name:
cx4.rom
KoF95.bin

Examples of New Name:
SNES_cx4.rom
SAT_KoF95.bin

Also changed the MSX 1.0 BIOS and BASIC vs MSX BIOS and BASIC have their own filenames now instead of a Filename Collision.

Check if completed:

Renaming Firmware Files based on System Name as BizHawk uses or at least renamed to be related to what it's for.
@hegyak hegyak added App: EmuHawk Relating to EmuHawk frontend Meta Relating to code organisation or to things that aren't code Core: MSXHawk MSX core labels Sep 7, 2023
@YoshiRulz

This comment was marked as resolved.

@YoshiRulz YoshiRulz removed the Core: MSXHawk MSX core label Sep 7, 2023
PC-Engine, Vectrex, Lynx, MSX And resolve Name Collision, Odyssey 2 G7400, Gameboy, GameBoy Color.
@hegyak
Copy link
Contributor Author

hegyak commented Sep 7, 2023

I do not get why the Commit I made is changing the Line Endings. But I have redone the changes.
They are:
PC-Engine, Vectrex, Lynx, MSX And resolve Name Collision, Odyssey 2 G7400, Gameboy, GameBoy Color.

That unifies the names for all the Firmware.

Comment on lines +113 to +114
FirmwareAndOption("79F5FF55DD10187C7FD7B8DAAB0B3FFBD1F56A2C", 262144, "PCECD", "Bios", "NEC_PCECD_3.0-(J).pce", "Super CD Bios (J)");
FirmwareAndOption("014881A959E045E00F4DB8F52955200865D40280", 32768, "PCECD", "GE-Bios", "NEC_PCECD_gecard.pce", "Games Express CD Card (Japan)");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NEC is a manufacturer name, not a console name.

Comment on lines +191 to +193
FirmwareAndOption("409E82ADAC40F6BDD18EB6C84E8B2FBDC7FB5498", 32768, "MSX", "bios_basic_usa", "Microsoft_MSX.rom", "MSX BIOS and BASIC");
FirmwareAndOption("3656BB3BBC17D280D2016FE4F6FF3CDED3082A41", 32768, "MSX", "bios_basic_usa", "Microsoft_MSX_1.0.rom", "MSX 1.0 BIOS and BASIC");
FirmwareAndOption("302AFB5D8BE26C758309CA3DF611AE69CCED2821", 32768, "MSX", "bios_basic_jpn", "Microsoft_MSX_jpn.rom", "MSX 1.0 JPN BIOS and BASIC");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Microsoft is not a part of the device name.

@YoshiRulz
Copy link
Member

I'd forgotten about #2204, but this is a follow-up to that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: EmuHawk Relating to EmuHawk frontend Meta Relating to code organisation or to things that aren't code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants