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

typo in '-o, --output TEMPLATE' %(stitle)s should be %(title)s #220

Closed
non7top opened this issue Nov 20, 2011 · 4 comments
Closed

typo in '-o, --output TEMPLATE' %(stitle)s should be %(title)s #220

non7top opened this issue Nov 20, 2011 · 4 comments

Comments

@non7top
Copy link

@non7top non7top commented Nov 20, 2011

Not quite sure how it was intended, but %(stitle)s doesn't ouput anything for me, and %(title)s outputs literal title.

@phihag
Copy link
Contributor

@phihag phihag commented Nov 20, 2011

$(stitle)s is the simplified or safe title. Since the original video title can contain characters such as slashes or backslashes, you almost certainly want stitle. If stitle is empty, but title is not, and the video's title is readable and safe on all platforms, this is a bug in the title simplification code. Can you post the exact command line that shows the problem?

It works fine for me:

$ youtube-dl -o 'X%(stitle)sY' http://www.youtube.com/watch?v=uHlDtZ6Oc3s
[youtube] Setting language
[youtube] uHlDtZ6Oc3s: Downloading video webpage
[youtube] uHlDtZ6Oc3s: Downloading video info webpage
[youtube] uHlDtZ6Oc3s: Extracting video information
[download] Destination: Xyoutube_dl_testvidY
@non7top
Copy link
Author

@non7top non7top commented Nov 20, 2011

It could be related to UTF filenames (cyrillic)

$ youtube-dl https://www.youtube.com/user/UniversalPicturesRus --format 37/22/18 --no-overwrites --quiet --output '/mnt/storage/download/%(autonumber)s-%(stitle)s.%(ext)s' --get-filename
/mnt/storage/download/00001-.mp4
/mnt/storage/download/00002-.mp4

$ youtube-dl https://www.youtube.com/user/UniversalPicturesRus --format 37/22/18 --no-overwrites --quiet --output '/mnt/storage/download/%(autonumber)s-%(title)s.%(ext)s' --get-filename
/mnt/storage/download/00001-Схватка. Русский трейлер.mp4
/mnt/storage/download/00002-Американский пирог. Все в сборе. Русский трейлер.mp4

@phihag
Copy link
Contributor

@phihag phihag commented Nov 20, 2011

I'll have a look at the simplification code. It should be generalized across all IEs anyways.

@phihag phihag closed this in af8e8d6 Nov 21, 2011
@phihag
Copy link
Contributor

@phihag phihag commented Nov 21, 2011

This should be fixed in youtube-dl 2011.11.21.

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
2 participants
You can’t perform that action at this time.