Opened 13 years ago

Closed 13 years ago

Last modified 13 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:
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 fingolfin, 13 years ago

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

comment:2 by fingolfin, 13 years ago

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 by fingolfin, 13 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 sev-, 13 years ago

Priority: normalblocker

comment:5 by sev-, 13 years ago

Raising priority. This is a release-critical bug.

comment:6 by eriktorbjorn, 13 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 salty-horse, 13 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 Kirben, 13 years ago

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

comment:9 by Kirben, 13 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 digitall, 13 months ago

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