diff --git a/src/lib/Client/Tools/POSIX.py b/src/lib/Client/Tools/POSIX.py
a
|
b
|
|
631 | 631 | return False |
632 | 632 | |
633 | 633 | # If we get here, then the parent directory should exist |
634 | | if (entry.get("paranoid", False) == 'true') and \ |
| 634 | if (entry.get("paranoid", False) in ('true', 'True')) and \ |
635 | 635 | self.setup.get("paranoid", False) and not \ |
636 | 636 | (entry.get('current_exists', 'true') == 'false'): |
637 | 637 | bkupnam = entry.get('name').replace('/', '_') |
… |
… |
|
639 | 639 | bkuplist = [f for f in os.listdir(self.ppath) if |
640 | 640 | f.startswith(bkupnam)] |
641 | 641 | bkuplist.sort() |
642 | | if len(bkuplist) == int(self.max_copies): |
| 642 | while len(bkuplist) >= int(self.max_copies): |
643 | 643 | # remove the oldest backup available |
644 | 644 | oldest = bkuplist.pop(0) |
645 | 645 | self.logger.info("Removing %s" % oldest) |
… |
… |
|
652 | 652 | try: |
653 | 653 | # backup existing file |
654 | 654 | shutil.copy(entry.get('name'), |
655 | | "%s/%s_%s" % (self.ppath, bkupnam, datetime.now())) |
| 655 | "%s/%s_%s" % (self.ppath, bkupnam, \ |
| 656 | datetime.isoformat(datetime.now()))) |
656 | 657 | self.logger.info("Backup of %s saved to %s" % |
657 | 658 | (entry.get('name'), self.ppath)) |
658 | 659 | except IOError, e: |