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

WIP: Extended ini support #126

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Conversation

ZashIn
Copy link
Contributor

@ZashIn ZashIn commented Oct 21, 2023

Extends ini support with optional:

  • [Features]
    • LocalSavegames = True (defaults to game.savesDirectory()) or custom path
    • SaveGamePreview = path to img
      • Add %SAVE_NAME% variable for e.g. %SAVE_NAME%.png
    • ForceLoadLibraries = list of .dll files
  • [BasicModDataChecker] and [BasicModDataChecker.move]
  • uses ExtendedInterpolation to support e.h. {BasicGames.key} substitution
  • Add BasicGames.GameSaveAdditionalFiles as simple option for ISaveGame.allFiles, with %SAVE_NAME% var
  • Add subfolder globbing */*.ext to BasicModDataChecker (requires larger changes in another PR)
  • Add template .ini file: game_template.ini

This would allow users without python knowledge to add a game support with some basic features or provide a good base for devs to add required functionality in python.

@Holt59
Copy link
Member

Holt59 commented Oct 21, 2023

I don't think that PR is very useful. Nobody has ever used the INI version of the game, so this is just adding complicated stuff to maintain for not much benefits.

Non-developers do not understand half of the entry in the INI, so adding more is not very useful, actual developers can easily write an equivalent Python plugin.

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