Incorrect use of "Days to keep" parameter in backup

[jgonzalez@…]

The "Days to keep" backup parameter is incorrectly used as an rdiff-backup parameter. This parameter is obtained in /etc/cron.daily/ebox-ebackup-cron in the following line:

my $keep = $model->backupKeepValue();

This is passed to /usr/share/ebox-ebackup/ebox-ebackup-local and then used as follows:

...
MAXAGE=$2
...
rdiff-backup --remove-older-than $MAXAGE --force $BACKUP/ > $LOG 2> $LOG
...

I think this causes rdiff-backup to take this value as an absolute date instead of a relative amount of days. The correct syntax to specify a number of days to keep is to add a capital D after the amount of days:

rdiff-backup --remove-older-than ${MAXAGE}D --force $BACKUP/ > $LOG 2> $LOG

[jacalvo@…]

Already fixed in ebox-ebackup 1.2.2