Skip to content

Fix Backup Manager Directory Issue and Build Script Output Name - #65

Merged
JacksonTheMaster merged 2 commits into
mainfrom
backup-fix
Aug 1, 2025
Merged

Fix Backup Manager Directory Issue and Build Script Output Name#65
JacksonTheMaster merged 2 commits into
mainfrom
backup-fix

Conversation

@JacksonTheMaster

@JacksonTheMaster JacksonTheMaster commented Aug 1, 2025

Copy link
Copy Markdown
Member

Description

This pull request addresses an issue where the backup manager was not restoring files to the correct directory (saves/Moon/Backup instead of saves/Moon). The fix modifies the RestoreBackup function to use the correct destination path. Additionally, it includes a correction to the build script's output file naming convention and bumps the version number.

Changes

  • src/backupmgr/restore.go:
    • Updated destFile path to use ./saves/"+config.WorldName instead of m.config.BackupDir to ensure backups are restored to the correct directory (saves/<WorldName>).

Further changes

  • Added debug logging for restoredFiles to improve traceability.
  • build/build.go:
    • Corrected output file name format by adding a v prefix to StationeersServerControl (e.g., StationeersServerControlv5.4.34 instead of StationeersServerControl5.4.34) for consistency.
  • src/config/process_config.go:
    • Bumped version from 5.4.33 to 5.4.34

Testing

  • Verified that backups now restore correctly to saves/<WorldName> (e.g., saves/Moon/world.xml).
  • Ran restore operation with index 7; files world_meta.xml, world.xml, and world.bin appear in the correct directory now

@JacksonTheMaster JacksonTheMaster left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@JacksonTheMaster
JacksonTheMaster merged commit 1a37529 into main Aug 1, 2025
1 check passed
@JacksonTheMaster
JacksonTheMaster deleted the backup-fix branch August 2, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant