AutoMySQLBackup / Bugs / #14 Weekly Rotation broken

#14 Weekly Rotation broken

Status: closed

Owner: nobody

Labels: None

Priority: 7

Updated: 2010-07-05

Created: 2006-10-11

Creator: John Sivak

Private: No

I just noticed that the Weekly backups weren't being
deleted after 5 weeks.

The reason is on line 539 of the script:
eval rm -fv "$BACKUPDIR/weekly/$DB_week.$REMW.*"

The line needs to be:
eval rm -fv "$BACKUPDIR/weekly/$DB/${DB}_week.$REMW.*"

Notice the extra $DB directory and the {}'s around the
$DB reference in the file name.

Not sure how this bug hasn't been hit yet..
I'm using separate backup files for each database.

Discussion

John Sivak - 2006-10-11

priority: 5 --> 7
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Thomas Kofler - 2006-10-18

Logged In: YES
user_id=142624

Confirm that bug

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

John Sivak - 2006-10-19

Logged In: YES
user_id=679094

Patch provided: Patch # 1580154 at
https://sourceforge.net/tracker/index.php?func=detail&aid=1580154&group_id=101066&atid=628966

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nobody/Anonymous - 2007-02-02

Logged In: NO

Just got hit with this one myself as I noticed my backup tarballs were getting larger every day. Surprised it wasn't mentioned earlier as a bug...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

tothborn - 2007-04-11

Logged In: YES
user_id=1245019
Originator: NO

You are right. Thank you for the submit.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nobody/Anonymous - 2007-10-21

Logged In: NO

Confirmed. Thank you.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Lehman - 2008-08-01

Logged In: YES
user_id=1910395
Originator: NO

Confirmed. Patch appears correct.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

budacsik - 2009-04-16

How is it necessary to use this patch?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Johannes Kolter - 2010-07-05

Changed routine for rotation.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Johannes Kolter - 2010-07-05

status: open --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.