jtmoon79
commented
5 years ago Seeing a similar problem. I've found if I pass any of the following options then the LastWriteTime property (according to powershell) or the Date modified column (within Windows Explorer) is not updated to the video's original release time: --xattrs, --add-metadata, or --embed-thumbnail.
With any of those options (--xattrs, --add-metadata, or --embed-thumbnail) passed to youtube-dl, the datetime is mistakenly set to the datetime of when writing to the file is completed. Without those options, the LastWriteTime (or Date modified) is set to the youtube video's Published datetime.
Successful time set
Here's an example of the success case: (LastWriteTime is set to video's Published datetime).
PS> C:\Python\Python37\Scripts\youtube-dl -o "%(upload_date)s %(title)s (%(id)s).%(ext)s" 2AFq2rti3-4
[youtube] 2AFq2rti3-4: Downloading webpage
[youtube] 2AFq2rti3-4: Downloading video info webpage
[download] Destination: 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f137.mp4
[download] 100% of 13.69MiB in 00:01
[download] Destination: 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f140.m4a
[download] 100% of 862.13KiB in 00:00
[ffmpeg] Merging formats into "20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).mp4"
Deleting original file 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f137.mp4 (pass -k to keep)
Deleting original file 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f140.m4a (pass -k to keep)
PS> ls
Mode LastWriteTime Length Name
---- ------------- ------ ----
-a---- 2019-02-11 15:26 15261624 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).mp4Failed time set
Here's an example of the failure case: (LastWriteTime is not set to video's Published datetime).
PS> C:\Python\Python37\Scripts\youtube-dl --xattrs -o "%(upload_date)s %(title)s (%(id)s).%(ext)s" 2AFq2rti3-4
[youtube] 2AFq2rti3-4: Downloading webpage
[youtube] 2AFq2rti3-4: Downloading video info webpage
[download] Destination: 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f137.mp4
[download] 100% of 13.69MiB in 00:01
[download] Destination: 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f140.m4a
[download] 100% of 862.13KiB in 00:00
[ffmpeg] Merging formats into "20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).mp4"
Deleting original file 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f137.mp4 (pass -k to keep)
Deleting original file 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).f140.m4a (pass -k to keep)
[metadata] Writing metadata to file's xattrs
PS> ls
Mode LastWriteTime Length Name
---- ------------- ------ ----
-a---- 2019-02-16 19:29 15261624 20190211 NASA Finds Second Massive Greenland Crater (2AFq2rti3-4).mp4Versions and settings
PS> C:\Python\Python37\Scripts\youtube-dl -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2019.02.08
[debug] Python version 3.7.1 (CPython) - Windows-10
[debug] exe versions: ffmpeg N-93129-g9e1e521393, ffprobe N-93129-g9e1e521393
[debug] Proxy map: {}In my case, the powershell instance's $env:PATH variable includes the paths to ffmpeg.exe (version 20190215-9e1e521-win64-static) and AtomicParsley.exe (version win32-0.9.0).
diegocr
MrDOS
Please follow the guide below
xinto all the boxes [ ] relevant to your issue (like this:[x])Make sure you are using the latest version: run
youtube-dl --versionand ensure your version is 2019.01.17. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.Before submitting an issue make sure you have:
What is the purpose of your issue?
If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the
-vflag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):Description of your issue, suggested solution and other information
When downloading a video, the default behaviour of youtube-dl is to set the modification time (mtime) of the output file to the time indicated by the Last-Modified response header of... something. When downloading multiple streams, it isn't clear which modification time is used. Regardless, this works fine, even when combining multiple streams:
(The video was originally uploaded in 2009, but the incorrect date appears to be the fault of YouTube, not youtube-dl.)
youtube-dl includes a handy feature,
--embed-thumbnail, which additionally retrieves a JPEG thumbnail for the video and embeds it into the output file. Unfortunately, this step is performed via an external tool (AtomicParsley), and happens after youtube-dl backdates the mtime of the output file. This means that the final mtime of the output file is ultimately set to the current time (by the system, because AtomicParsley modifies the file), not the modification time of the video as indicated by the source.I suspect the correct fix here is to perform mtime adjustment of the output file at the very last minute, after all other possible options applying to the single output file have been handled.