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

Read enabled .mod files from ini file #285

Merged
merged 5 commits into from
Aug 22, 2022
Merged

Read enabled .mod files from ini file #285

merged 5 commits into from
Aug 22, 2022

Conversation

matthiakl
Copy link
Contributor

Currently all files in Data/ and Data/modvdf are loaded. This prevents having multiple mods installed. With this change, only the .mod files listed in the ini file (with -game:<inifile>) are loaded.

[FILES]
vdf=FunnyMod.mod (Gothic 1)
VDF=FunnyMod.mod (Gothic 2)
List of all .mod files created for the mod. If this is more than one, they are listed separated by spaces (FunnyMod1.mod FunnyMod2.mod ...). GothicStarter first copies all *.mod files from .../Data/ to .../Data/modvdf/ before starting a modification and then copies all specified files from .../Data/modvdf/ to .../Data. The value "VDF" must exist (even if it is empty) for GothicStarter to accept the INI. In G2, the special value "!" can be used to explicitly disable reading by the GothicStarter. During development a VDF list is most useful. It is very important with G2 that the parameter GohticGame.mod is not here! This leads to the fact that all changes to the scripts are ignored.

https://wiki.worldofgothic.de/doku.php?id=ini-datei

@matthiakl
Copy link
Contributor Author

@Try
Copy link
Owner

Try commented Aug 21, 2022

In error log:

instantiation of undefined template 'std::ctype<char16_t>'

It seems MacOS doesn't support ctype for anything else than a char. so using std::getline is not an option. Generally it woruld be nice to avoid basic_stringstream, completely

game/gothic.cpp Show resolved Hide resolved
game/gothic.cpp Outdated Show resolved Hide resolved
game/resources.cpp Show resolved Hide resolved
@Try Try merged commit 2af6279 into Try:master Aug 22, 2022
@Try
Copy link
Owner

Try commented Aug 22, 2022

Merged, thanks for PR!

@matthiakl matthiakl deleted the vdflist branch August 22, 2022 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants