Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

youtube-dl using wrong file names internally for video with Russian title #2225

Closed
hurda opened this issue Jan 24, 2014 · 4 comments
Closed

youtube-dl using wrong file names internally for video with Russian title #2225

hurda opened this issue Jan 24, 2014 · 4 comments

Comments

@hurda
Copy link
Contributor

@hurda hurda commented Jan 24, 2014

youtube-dl version 2014.01.23.4

URL: https://www.youtube.com/watch?v=8ke_HEaWrLY

Console-log:

C:\Users\user\Desktop>youtube-dl.exe -f 134+140 https://www.youtube.com/watch?v=8ke_HEaWrLY
[youtube] Setting language
[youtube] 8ke_HEaWrLY: Downloading webpage
[youtube] 8ke_HEaWrLY: Downloading video info webpage
[youtube] 8ke_HEaWrLY: Extracting video information
[download] Destination:   ,  -8ke_HEaWrLY.f134.mp4
[download] 100% of 1.37MiB in 00:03
[download] Destination:   ,  -8ke_HEaWrLY.f140.m4a
[download] 100% of 586.57KiB in 00:01
[ffmpeg] Merging formats into "  ,  -8ke_HEaWrLY.mp4"
ERROR:   ,  -8ke_HEaWrLY.f134.mp4: No such file or directory

But the names of the downloaded files are Супер Быстрая Девочка, Нереальная Скорость-8ke_HEaWrLY.f134.mp4 and Супер Быстрая Девочка, Нереальная Скорость-8ke_HEaWrLY.f140.m4a.

It's working with --restrict-filenames, but it shouldn't be necessary.

PS: I know -f 134+140 isn't necessary for this video, it was just a test.

EDIT:

C:\Users\user\Desktop>youtube-dl.exe -f 134+140 -v https://www.youtube.com/watch?v=8ke_HEaWrLY
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-f', '134+140', '-v', 'https://www.youtube.com/watch?v=8ke_HEaWrLY']
[debug] Encodings: locale 'cp1252', fs 'mbcs', out 'cp850', pref: 'cp1252'
[debug] youtube-dl version 2014.01.23.4
[debug] Python version 2.7.5 - Windows-7-6.1.7601-SP1
[debug] Proxy map: {}
[youtube] Setting language
[youtube] 8ke_HEaWrLY: Downloading webpage
[youtube] 8ke_HEaWrLY: Downloading video info webpage
[youtube] 8ke_HEaWrLY: Extracting video information
[download] Destination:   ,  -8ke_HEaWrLY.f134.mp4
[download] 100% of 1.37MiB in 00:02
[download] Destination:   ,  -8ke_HEaWrLY.f140.m4a
[download] 100% of 586.57KiB in 00:01
[ffmpeg] Merging formats into "  ,  -8ke_HEaWrLY.mp4"
[debug] ffmpeg command line: ffmpeg -y -i '  ,  -8ke_HEaWrLY.f134.mp4' -i '  ,
-8ke_HEaWrLY.f140.m4a' -c copy '  ,  -8ke_HEaWrLY.mp4'
ERROR:   ,  -8ke_HEaWrLY.f134.mp4: No such file or directory
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 1016, in post_process
  File "youtube_dl\postprocessor\ffmpeg.pyo", line 483, in run
  File "youtube_dl\postprocessor\ffmpeg.pyo", line 61, in run_ffmpeg_multiple_fi
les
FFmpegPostProcessorError
@hurda
Copy link
Contributor Author

@hurda hurda commented Jan 24, 2014

Switching cmd's codepage to 65001 (utf8) has no effect:

C:\Users\user\Desktop>youtube-dl.exe -s --get-filename -v https://www.youtube.com/watch?v=8ke_HEaWrLY
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-s', '--get-filename', '-v', 'https://www.youtube.com/watch?v=8ke_HEaWrLY']
[debug] Encodings: locale 'cp1252', fs 'mbcs', out 'cp65001', pref: 'cp1252'
[debug] youtube-dl version 2014.01.23.4
[debug] Python version 2.7.5 - Windows-7-6.1.7601-SP1
[debug] Proxy map: {}
  ,  -8ke_HEaWrLY.mp4
@ivan
Copy link
Contributor

@ivan ivan commented Nov 20, 2014

This should work when running youtube-dl with Python 3.4.

Python 2.x's subprocess doesn't really work with Unicode on Windows (http://bugs.python.org/issue19264). I used to use 8.3 filenames to work around this, but it doesn't work for .webm files. Also, 8.3 filenames are being phased out in Windows 8.1 for performance reasons.

@hurda
Copy link
Contributor Author

@hurda hurda commented Nov 20, 2014

The bug happens with the compiled executable.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jun 13, 2016

This was fixed with switch to python 3.4 for Windows build.

@dstftw dstftw closed this Jun 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.