mtkennerly / ludusavi

Backup tool for PC game saves
MIT License
2.33k stars 54 forks source link

ludusavi backup requires to specify --merge although it is set in config.yaml #133

Closed sluedecke closed 1 year ago

sluedecke commented 1 year ago

Ludusavi version

v0.13.1

Operating system

Linux

Installation method

Other

Description

$ ludusavi --version
ludusavi 0.13.1
$ ludusavi  backup Factorio
The backup target already exists ( /home/MYUSER/Games/00-savegames/0-ludusavi-backup ). Either choose a different --path or delete it with --force.
$ ludusavi  backup --merge Factorio
Factorio [187.88 MiB]:
  - /home/MYUSER/Games/Factorio/game/config/config.ini
  - /home/MYUSER/Games/Factorio/game/saves/_autosave1.zip
  - /home/MYUSER/Games/Factorio/game/saves/_autosave2.zip
[...]

Overall:
  Games: 1
  Size: 187.88 MiB
  Location: /home/MYUSER/Games/00-savegames/0-ludusavi-backup

Ludusavi configuration:

image

Logs

No response

mtkennerly commented 1 year ago

Yeah, this isn't a good experience. I think it should prompt for confirmation, like it does in the GUI, unless you use --force to bypass that. The phrasing of the error is a carryover from before the merge option was available.