diff options
| author | elyang0214 <elyang@umich.edu> | 2023-04-02 11:14:52 -0400 |
|---|---|---|
| committer | elyang0214 <elyang@umich.edu> | 2023-04-02 11:14:52 -0400 |
| commit | 09de7909ce129112f5aa75167aec51709f727f02 (patch) | |
| tree | c69a9e235e6324043b8209e334eadadfecbd692d | |
| parent | b3b26efe88bf3dc41bacd4fbc27249a3cd6352e7 (diff) | |
| parent | fbd665a41720b22ac7521a5aa05c3a28f5331b75 (diff) | |
Merge branch 'master' of https://github.com/elyang0214/beets
| -rw-r--r-- | beets/importer.py | 13 | ||||
| -rw-r--r-- | docs/changelog.rst | 2 |
2 files changed, 15 insertions, 0 deletions
diff --git a/beets/importer.py b/beets/importer.py index feebadc09..96d1f17df 100644 --- a/beets/importer.py +++ b/beets/importer.py @@ -1122,6 +1122,19 @@ class ArchiveImportTask(SentinelImportTask): archive = handler_class(util.py3_path(self.toppath), mode='r') try: archive.extractall(extract_to) + + # Adjust the files' mtimes to match the information from the + # archive. Inspired by: https://stackoverflow.com/q/9813243 + for f in archive.infolist(): + # The date_time will need to adjusted otherwise + # the item will have the current date_time of extraction. + # The (0, 0, -1) is added to date_time because the + # function time.mktime expects a 9-element tuple. + # The -1 indicates that the DST flag is unknown. + date_time = time.mktime(f.date_time + (0, 0, -1)) + fullpath = os.path.join(extract_to, f.filename) + os.utime(fullpath, (date_time, date_time)) + finally: archive.close() self.extracted = True diff --git a/docs/changelog.rst b/docs/changelog.rst index 6aa2c7302..77049e800 100644 --- a/docs/changelog.rst +++ b/docs/changelog.rst @@ -54,6 +54,8 @@ New features: * :ref:`import-options`: Add support for re-running the importer on paths in log files that were created with the ``-l`` (or ``--logfile``) argument. :bug:`4379` :bug:`4387` +* Preserve mtimes from archives + :bug:`4392` * Add :ref:`%sunique{} <sunique>` template to disambiguate between singletons. :bug:`4438` * Add a new ``import.ignored_alias_types`` config option to allow for |