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

Please update the default_roms_path to follow macOS Standards #29

Open
d3vi1 opened this issue Dec 22, 2021 · 2 comments
Open

Please update the default_roms_path to follow macOS Standards #29

d3vi1 opened this issue Dec 22, 2021 · 2 comments

Comments

@d3vi1
Copy link

d3vi1 commented Dec 22, 2021

Hi,

The default ROMS behavior should be changed. On the application startup, a folder ~/Library/Application Support/PCem should be created and the default roms,configs,nvr, etc. should be copied from $executable_path/../Resources/SKEL to that folder, so that the user can customize them without actually modifying the /Applications/PCem.app binary. If that initial provisioning is done, PCem shouldn't complain about ROMS at startup anymore. The expected path of ~/PCem or /usr/share/pcem is not documented in the error dialog box nor in stderr, nor is it created automatically.

Thanks,
R.

@almeath
Copy link
Member

almeath commented Dec 22, 2021

Should this request be lodged in the bundler repository? (https://github.com/PCemOnMac/bundler)

The binary already looks for the "roms" folder (and all other config folders) under Application Support/PCem by default, as of this commit:

a06d30a

There still has be be valid roms inside the roms folder in order to avoid the error message, and those cannot be included by default due to copyright. I just tested this now with a fresh build and it is working fine as long as valid roms go in the said folder.

If the problem is being separately caused by the behavior of the PCem application bundle, I will let @kyr0 know about it, as he has maintained the bundler going back to PCem v14.

@almeath
Copy link
Member

almeath commented Dec 22, 2021

I have mentioned in issue #28 that I am working on my own application bundle for PCem (separate to the one developed by @kyr0 ). If I can get issue #28 resolved, I can at least provide that bundle for download, which does create the config folders under Application Support/PCem.

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

No branches or pull requests

2 participants