Allow to pass stat-timestamp in config

Signed-off-by: Olivier Mehani <shtrom@ssji.net>

Author: shtrom

rotate_backups/__init__.py

@@ -228,6 +228,8 @@ def load_config_file(configuration_file=None, expand=True):
         # 'timestamp-pattern' configuration file option has a value set.
         if items.get('timestamp-pattern'):
             options['timestamp_pattern'] = items['timestamp-pattern']
+        if items.get('stat-timestamp'):
+            options['stat_timestamp'] = items['stat-timestamp']
         # Expand filename patterns?
         if expand and location.have_wildcards:
             logger.verbose("Expanding filename pattern %s on %s ..", location.directory, location.context)
@@ -404,9 +406,6 @@ def __init__(self, rotation_scheme, **options):
         """
         options.update(rotation_scheme=rotation_scheme)
         super(RotateBackups, self).__init__(**options)
-        if self.stat_timestamp:
-            logger.info("Using file mtime to determine file date")
-            self.matcher = FilestatMatcher()
 
     @mutable_property
     def config_file(self):
@@ -550,6 +549,15 @@ def rotation_scheme(self):
     @mutable_property
     def stat_timestamp(self):
         """Whether to use the files' mtime instead of parsing their name."""
+        return isinstance(self.match, FilestatMatcher)
+
+    @stat_timestamp.setter
+    def stat_timestamp(self, value):
+        if value:
+            logger.info("Using file mtime to determine file date")
+            self.matcher = FilestatMatcher()
+        elif isinstance(self.match, FilestatMatcher):
+            del self.matcher
 
     @mutable_property
     def strict(self):

rotate_backups/tests.py

@@ -258,12 +258,12 @@ def test_rotate_backups_mtime(self):
             parser.set(subdir, 'monthly', '12')
             parser.set(subdir, 'yearly', 'always')
             parser.set(subdir, 'ionice', 'idle')
+            parser.set(subdir, 'stat-timestamp', 'yes')
             with open(config_file, 'w') as handle:
                 parser.write(handle)
             self.create_sample_backup_set(root)
             map = self.apply_mtime_and_rename(root, subdir)
-            run_cli(main, '--verbose', '--config=%s' % config_file,
-                    '--stat-timestamp')
+            run_cli(main, '--verbose', '--config=%s' % config_file)
             backups_that_were_preserved = set(os.listdir(subdir))
             assert backups_that_were_preserved == set([map[e]
                                                        for e in expected_to_be_preserved])