[BackupScreen] Errors from non-existent files

Issue #601 resolved
prl created an issue

The old implementation of BackupScreen removed non-existent files too early. The current implementation tries to back up all paths in config.plugins.configurationbackup.backupdirs, even if they don't exist, resulting in unnecessary error messages.

Comments (2)

  1. Peter Urbanec

    Fix Bug #601: [BackupScreen] Errors from non-existent files

    "glob" the paths in config.plugins.configurationbackup.backupdirs and eliminate non-existant files from the list of files to be backed up.

    Filenames containing whitespace and other non-glob shell special characters will now also be corectly handled (except in now-unused self.backupdirs).

    The command now no longer needs to be run via a shell invocation, so use the exec() form of Console execution instead of the system() style.

    Retain self.backupdirs as a space-separated list of file paths for backward compatibility.

    → <<cset 9fda5bebb2c9>>

  2. Log in to comment