The --output argument is messed up... #407

Closed
opened 2026-02-20 23:03:11 -05:00 by deekerman · 2 comments
Owner

Originally created by @sajal on GitHub (Nov 27, 2012).

Originally assigned to: @phihag on GitHub.

Running

/tmp/youtube-dl http://www.youtube.com/watch?v=715qvhao_y0 -f 18 -o "/mnt/foo"

I expect the output to be stored in /mnt/foo . But the latest master stores it as filename "mnt_foo" in the current directory. Please advise. Do i need to escape the slash?

The problem started when i updated the script few hours ago. I had updated it yesterday as well, no problems then.

Originally created by @sajal on GitHub (Nov 27, 2012). Originally assigned to: @phihag on GitHub. Running ``` /tmp/youtube-dl http://www.youtube.com/watch?v=715qvhao_y0 -f 18 -o "/mnt/foo" ``` I expect the output to be stored in /mnt/foo . But the latest master stores it as filename "mnt_foo" in the current directory. Please advise. Do i need to escape the slash? The problem started when i updated the script few hours ago. I had updated it yesterday as well, no problems then.
Author
Owner

@FiloSottile commented on GitHub (Nov 27, 2012):

Ugh. @phihag people were using -o to specify folders, and now we escape also user-supplied /s. I think this was accepted, valid and widespread behavior!

We should rollback to escaping every info_dict field.

@FiloSottile commented on GitHub (Nov 27, 2012): Ugh. @phihag people were using `-o` to specify folders, and now we escape also user-supplied `/`s. I think this was accepted, valid and **widespread** behavior! We should rollback to escaping every `info_dict` field.
Author
Owner

@phihag commented on GitHub (Nov 27, 2012):

Sorry, that was indeed a fault on my part. It's fixed now, and it has been fixed in youtube-dl 2012.11.28.

@phihag commented on GitHub (Nov 27, 2012): Sorry, that was indeed a fault on my part. It's fixed now, and it has been fixed in youtube-dl 2012.11.28.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/youtube-dl-ytdl-org#407
No description provided.