Opened 12 years ago

Closed 12 years ago

Last modified 7 months ago

#3169 closed defect (fixed)

WIN32: Document the windows config file location change

Reported by: salty-horse Owned by: Kirben
Priority: blocker Component: Port: Win32
Keywords: Cc:


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\".


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 Changed 12 years ago by fingolfin

Owner: set to Kirben
Summary: Document the windows config file location changeWIN32: Document the windows config file location change

comment:2 Changed 12 years ago by fingolfin

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.

comment:3 Changed 12 years ago by fingolfin

I.e. essentially, section 8.0) Configuration file: in the README needs to be updated (the fallback mechanism should be documented, too).

comment:4 Changed 12 years ago by sev-

Priority: normalblocker

comment:5 Changed 12 years ago by sev-

Raising priority. This is a release-critical bug.

comment:6 Changed 12 years ago by eriktorbjorn

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 Changed 12 years ago by salty-horse

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 Changed 12 years ago by Kirben

Component: --Unset--
Resolution: fixed
Status: newclosed

comment:9 Changed 12 years ago by Kirben

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 Changed 7 months ago by digitall

Component: --Unset--Port: Win32
Note: See TracTickets for help on using tickets.