#3712 ALL: --savepath=PATH broken

[Marek Roth]

ScummVM 0.12.0svn (Apr 30 2008 10:27:14)
Features compiled in: Vorbis FLAC MP3 zLib MPEG2
Windows 2000 SP4

The command line option --savepath=PATH doesn't seem to do anything on the svn build as well as in the later releases. 0.8.2 is the last release in which this options works.
This example does not work in releases after 0.8.2:
scummvm --config=scummvm.ini --savepath="%appdata%\scummvm"

[Max Horn]

Logged In: YES
user_id=12935
Originator: NO

It works great for me under Mac OS X.

Maybe a Windows specific problem. Kirben, can you confirm the issue?

Marek, can you try it with slashes instead of backslashes, just in case?

[kirben]

Logged In: YES
user_id=34715
Originator: NO

I can reproduce the problem, but only if I don't select a target name.

'scummvm --savepath=c:/tmp tentacle' uses the correct save path

'scummvm --savepath=c:/tmp' uses the default save path

The slashes aren't important, using forwardslash or backwardslash seems to work fine.

[Max Horn]

Logged In: YES
user_id=12935
Originator: NO

Thanks, I see now. This is indeed an old and complicated issue and I think it affects *all* command line settings, not just savepath. Supporting these leads to a certain clash with the options dialog.

Roughly, we don't want to accidentally overwrite the savepath set in the config file by the temporary savepath, which could/would happen when the user uses the options dialog.

[Adrian]

Logged In: YES
user_id=2095882
Originator: NO

This does not affect all command line options. I'm trying to use the following:
--savepath=%U3_APP_DATA_PATH%\Save --extrapath=%U3_DEVICE_EXEC_PATH% --themepath=%U3_DEVICE_EXEC_PATH% --config=%U3_APP_DATA_PATH%\scummvm.ini

The last 3 work fine however the --savepath parameter seems to be ignored when a config file is specified.

It seems like a bug that command line options override config file options EXCEPT for --savepath. What's strange, though, is that Scumm is CHECKING the --savepath specified parameter because if that folder does not exist ScummVM.exe complains and exits.

[Max Horn]

Well, this happens because launcherDialog() clears the kTransientDomain domain; and this should actually also affect the extrapath & themepath, only that it's less noticable there, since the themepath is not needed anymore once the theme has been loaded.

So, despite all, this is still a generic problem, a deep conflict caused by our attempt to simultaneously support a command line and a graphical interface.