09242: Source inifile name conflict

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
09242	Misc.	Minor	Sometimes	11 days ago	11 days ago

Tester	Robbbert	View Status	Public	Platform	MAME (Self-compiled)
Assigned To		Resolution	Open	OS	Windows 11/10 (64-bit)
Status [?]	Confirmed			Driver
Version	0.279	Fixed in Version		Build	64-bit
		Fixed in Git Commit		Github Pull Request #

Summary	09242: Source inifile name conflict
Description	With different sources now having the same name (see https://github.com/mamedev/mame/commit/cf8e2d692709533402a33d1f51fb94e83517fd17 for example), this means that if you make a source ini file for junior.cpp, it will apply to both cxg\junior.cpp and elektor\junior.cpp - the same problem for the other duplicate names. There's various solutions to this, of which I have no preference. 1. Remove support for source ini files - I doubt it's used very much; OR 2. ini \ source \ cxg \ junior.ini and ini \ source \ elektor \ junior.ini; OR 3. ini \cxg \ junior.ini and ini \ elektor \ junior.ini (in other words remove the hard-coded "source" subfolder). In any case, the documentation at https://docs.mamedev.org/advanced/multiconfig.html will need updating.
Steps To Reproduce
Additional Information	The mamedev habit of moving source files between various source folders will have an impact.
Github Commit

Flags
Regression Version	0.246
Affected Sets / Systems

Attached Files

No.23592 hap Developer 11 days ago	I'd say #2. It's in src/frontend/mame/mameopts.cpp There's one in emu/config.cpp too. I'm not sure what it's trying there, does it want to load "ctrl/sourcefile.cpp.cfg"?
No.23593 Robbbert Moderator 11 days ago	The config.cpp has nothing to do with this bug. Adding a display showed it was only trying to open default.cfg then gamename.cfg, as you'd expect. The "source" stuff seems to be something for controller configs.