Skip to content

Comments

Allow additional youtube-dl arguments#163

Merged
mxpv merged 1 commit intomxpv:masterfrom
ticky:additional-youtubedl-arguments
Aug 11, 2020
Merged

Allow additional youtube-dl arguments#163
mxpv merged 1 commit intomxpv:masterfrom
ticky:additional-youtubedl-arguments

Conversation

@ticky
Copy link
Contributor

@ticky ticky commented Aug 11, 2020

This allows inserting arbitrary extra arguments into the youtube-dl command-line within each podcast feed entry. This allows, for example, requesting that youtube-dl embed subtitles and captions into the video file, or configuring additional postprocessor arguments.

No effort has been made to make sure no incompatible arguments are provided, with the expectation that the end-user will understand what's possible, and that this is an option expressly for advanced users.

I’ve confirmed that the mp4 output from youtube-dl with the included example settings does indeed embed captions of whatever sort is available, and that the resultant video subtitles, of both sorts, then work in Apple Podcasts.

This would fix #162, and theoretically also #136!

This allows inserting arbitrary extra arguments into the youtube-dl command-line within each podcast feed entry. This allows, for example, requesting that youtube-dl embed subtitles and captions into the video file, or configuring additional postprocessor arguments.

No effort has been made to make sure no incompatible arguments are provided, with the expectation that the end-user will understand what's possible, and that this is an option expressly for advanced users.

This would fix mxpv#162,and theoretically also mxpv#136!
@ticky ticky mentioned this pull request Aug 11, 2020
@mxpv
Copy link
Owner

mxpv commented Aug 11, 2020

Nice! thanks!

@mxpv mxpv merged commit 303d741 into mxpv:master Aug 11, 2020
@ticky
Copy link
Contributor Author

ticky commented Aug 12, 2020

Awesome, thanks for the quick merge! 😃

@ticky
Copy link
Contributor Author

ticky commented Sep 8, 2020

Any chance of getting this in a release? I’d love to pop it into my Docker setup!

@mxpv
Copy link
Owner

mxpv commented Sep 9, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Embedding Subtitles and Captions

2 participants