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

ConfigManager: Sync Dolphin settings to SYSCONF on exit #4332

Merged
merged 1 commit into from Oct 14, 2016

Conversation

leoetlino
Copy link
Member

@leoetlino leoetlino commented Oct 9, 2016

#4319 made Dolphin not read/write directly to the SYSCONF and read settings from the SYSCONF at boot, and only write Dolphin settings to the SYSCONF at emulation startup.

However, this also made it a bit confusing, because if settings were changed, then Dolphin was exited without starting a game in between, the settings wouldn't actually get persisted. This is fixed by syncing Dolphin settings with the SYSCONF when Dolphin exits.


This change is Reviewable

4319 made Dolphin not read/write directly to the SYSCONF and read
settings from the SYSCONF at boot, and only write Dolphin settings
to the SYSCONF at emulation startup.

However, this also made it a bit confusing, because if settings were
changed, then Dolphin was exited without starting a game in between,
the settings wouldn't actually get persisted. This is fixed by
syncing Dolphin settings with the SYSCONF when Dolphin exits.
@psennermann
Copy link

For SYSCONF you mean the config.ini in user/documents/Dolphin or is there another configuration file?

@JosJuice
Copy link
Member

This is about the SYSCONF file in the NAND.

@psennermann
Copy link

And where do I find it? I'm asking because I think something got corrupted in my Wiimote config files and wanted to set it back to deafult...

@JosJuice
Copy link
Member

shared2/sys/SYSCONF (in the NAND)

@degasus degasus merged commit 183f3c3 into dolphin-emu:master Oct 14, 2016
@leoetlino leoetlino deleted the sync-sysconf-on-exit branch October 18, 2016 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants