Opened 18 years ago
Closed 17 years ago
#5975 closed patch (fixed)
mythconverg_restore.pl - exclude backup and restore scripts from list of backup files
| Reported by: | Owned by: | Rob Smith | |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | mythtv | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Now that the database backup and restore script filenames have been changed to start with mythconverg, they match the glob used to find backup files when a filename is not specified. In the event that a user puts the scripts in the DBBackupDirectory, one of the two scripts will be sent to the database as if it were a backup file, resulting in errors such as:
ERROR 1049 (42000) at line 11: Unknown database 'Getopt::Long'
or
ERROR 1044 (42000) at line 11: Access denied for user 'mythtv'@'localhost' to database 'Getopt::Long'
The attached patch simply adds a line that removes the script names from the list of files before sorting and finding a backup file to use.
Attachments (4)
Change History (7)
by , 18 years ago
| Attachment: | mythtv-5975-restore-exclude_script_from_file_selection.patch added |
|---|
by , 18 years ago
| Attachment: | mythtv-5975-restore-exclude_script_from_file_selection-20081206.patch added |
|---|
comment:1 by , 18 years ago
mythtv-5975-restore-exclude_script_from_file_selection-20081206.patch is an updated patch that also fixes a conditional. When rearranging verbose output, a single-line conditional was broken into two lines but only one was executed conditionally. This adds a proper conditional block.
Thanks to Jack M. for reporting the issues.
comment:2 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
by , 17 years ago
| Attachment: | mythtv-5975-restore-exclude_script_from_file_selection.2.patch added |
|---|
Updated patch for recent changes to trunk.
by , 17 years ago
| Attachment: | mythtv-5975_and_6013-combined.patch added |
|---|

Updated patch