#3169 closed defect (fixed)
WIN32: Document the windows config file location change
| Reported by: | salty-horse | Owned by: | Kirben |
|---|---|---|---|
| Priority: | blocker | Component: | Port: Win32 |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
Since the release of 0.9.0, the location of the configuration file on Windows was changed from c:\windows to the user-specific "Application Data\".
See http://svn.sourceforge.net/viewvc/scummvm?view=rev&revision=23465
While the migration from the old location to the new is done automatically when first running ScummVM, the README (and NEWS file) should be updated to mention the change.
Ticket imported from: #1709173. Ticket imported from: bugs/3169.
Change History (10)
comment:1 by , 17 years ago
| Owner: | set to |
|---|---|
| Summary: | Document the windows config file location change → WIN32: Document the windows config file location change |
comment:2 by , 17 years ago
comment:3 by , 17 years ago
I.e. essentially, section 8.0) Configuration file: in the README needs to be updated (the fallback mechanism should be documented, too).
comment:4 by , 17 years ago
| Priority: | normal → blocker |
|---|
comment:6 by , 17 years ago
ScummVM actually tells the user about the change of location, if it finds a config file in the old location. However, it does so by using printf(), so the number of Windows users that will actually notice this is probably pretty small.
See common/config-manager.cpp, somewhere around line 120 or so.
I'm not sure if we could use ScummVM's GUI to print a message at this stage - the config manager is initialized pretty early, is it not? - but since this is in a Windows-specific area, perhaps we could use a standard Windows message box instead?
comment:7 by , 17 years ago
I'm not sure the visibility of the runtime message is that important to the average user. What matters most is that it "just works".
comment:8 by , 17 years ago
| Component: | → --Unset-- |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:9 by , 17 years ago
I updated the default config locations in the README file and added a brief note to the NEWS file.
A GUI message would annoy or confuse users, especially if a user wanted to keep the previous default config file location. And a user doesn't need to take any manual action at all.
comment:10 by , 5 years ago
| Component: | --Unset-- → Port: Win32 |
|---|
Kirben, you made that change, please update the docs accordingly.
I don't think we need to document this in the NEWS file. After all, we automatically fallback to the old location anyway, so existing users should not notice the change.