garethgeorge / backrest

Backrest is a web UI and orchestrator for restic backup.
GNU General Public License v3.0
787 stars 31 forks source link

Minor improvement: --exclude-caches is in cmd twice #304

Open berti34 opened 1 month ago

berti34 commented 1 month ago

Hi George,

just a notice: the --exclude-caches is always twice in the backup cmd:

command: C:\WINDOWS\SYSTEM32\restic.exe backup --json --exclude-caches C:\Portable C:\Privat -o sftp.args=-oBatchMode=yes --exclude-caches --tag plan:S3-DTSMS-Win-01 --tag created-by:DTS-LAP-028 --exclude *~ --parent ... --exclude-file C:/Portable/BackRest/excludes-file.txt

berti34 commented 1 month ago

Do you think you may filter out some backup flags that are OS dependent? For example --one-file-system is currently not possible on Windows and throws an error during backup start. Maybe the configuration dialog should give a notice or warning about it, depending on the system the backrest binary is running on.