Skip to content

Add new config key RomsDirectory to Supermodel.ini - #331

Open
hywelandrews wants to merge 1 commit into
trzy:masterfrom
hywelandrews:new-config-rom-folder-path
Open

Add new config key RomsDirectory to Supermodel.ini#331
hywelandrews wants to merge 1 commit into
trzy:masterfrom
hywelandrews:new-config-rom-folder-path

Conversation

@hywelandrews

Copy link
Copy Markdown

Rational: When using Supermodel from it's GUI roms are currently searched only in a child ROMs/ folder. This is a problem in particular when Supermodel is installed from a system package manager (homebrew, pkg, apt etc). This change allows users to configure any path to be searched when launching roms from the GUI. It is not provided as a command line option, only as a key in Supermodel.ini.

Thanks

@hywelandrews hywelandrews changed the title Add new config key RomFolderPath to Supermodel.ini Add new config key RomsFolderPath to Supermodel.ini Aug 6, 2026
@trzy

trzy commented Aug 6, 2026

Copy link
Copy Markdown
Owner

I need to take a look at this because I thought we had a mechanism for locating directories according to (obtuse) UNIX conventions but that might’ve been only for the config file.

one nit: call it “directory”, not folder, to be consistent with the rest of the app.

@hywelandrews
hywelandrews force-pushed the new-config-rom-folder-path branch from 537a43e to 2506d63 Compare August 7, 2026 12:18
@hywelandrews hywelandrews changed the title Add new config key RomsFolderPath to Supermodel.ini Add new config key RomsDirectory to Supermodel.ini Aug 7, 2026
@hywelandrews

hywelandrews commented Aug 7, 2026

Copy link
Copy Markdown
Author

Sure, I've updated the config key and internal variables to [Rr]omsDirectory. I had a look at the OSD/FileSystem implementations when submitting the FreeBSD port, and all platforms did just have to handle config paths but I've only been able to test this on MacOSX and UNIX. Would also partially resolve #278

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.

2 participants