Skip to content

Update to fix saving issue with Dungeon Keeper Gold and Deeper Dungeon#555

Open
trotsky40 wants to merge 4 commits into
PureDOS:mainfrom
trotsky40:dungeon-keeper-update
Open

Update to fix saving issue with Dungeon Keeper Gold and Deeper Dungeon#555
trotsky40 wants to merge 4 commits into
PureDOS:mainfrom
trotsky40:dungeon-keeper-update

Conversation

@trotsky40
Copy link
Copy Markdown
Contributor

Added an empty "SAVE" folder to fix in-game saving and added cpu_max_year: 1998 to prevent occasional speed ups

Added an empty "SAVE" folder to fix in-game saving and added cpu_max_year: 1998 to prevent occasional speed ups
@trotsky40
Copy link
Copy Markdown
Contributor Author

Addresses #548

@PureDOS
Copy link
Copy Markdown
Owner

PureDOS commented May 1, 2026

Would it be better if the "SAVE" folder were added to the DAT?

@trotsky40
Copy link
Copy Markdown
Contributor Author

We didn’t think the DAT could generate an empty folder. If it can, I can rework the PR.

@PureDOS
Copy link
Copy Markdown
Owner

PureDOS commented May 10, 2026

An empty save folder can easily be added to the dat with

		<rom name="SAVE/" size="0" crc="00000000" md5="d41d8cd98f00b204e9800998ecf8427e" sha1="da39a3ee5e6b4b0d3255bfef95601890afd80709"/>

So having a / at the end of the name there means it defines a folder. This is used to create empty folders but also to have folders created with a specific date/time stamp if applicable (by also adding the date field). There's quite a few like these in the DAT file we have.

@trotsky40 trotsky40 marked this pull request as draft May 10, 2026 14:27
@trotsky40
Copy link
Copy Markdown
Contributor Author

Copy. I will rework it.

@trotsky40 trotsky40 marked this pull request as ready for review May 16, 2026 03:36
@trotsky40
Copy link
Copy Markdown
Contributor Author

Moved the save folder to the dat

@trotsky40 trotsky40 changed the title Updated DOSC to fix saving issue with Dungeon Keeper Gold and Deeper Dungeon Update to fix saving issue with Dungeon Keeper Gold and Deeper Dungeon May 16, 2026
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