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

add hls method to write ts files #12

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AndreaNicola
Copy link
Contributor

Greetings! I needed to create some m3u8 playlists. I added an Hls method to Video struct that inserts the necessary options for ffmpeg to do them.

@AndreaNicola
Copy link
Contributor Author

Hello! :-D is there anybody out there? :-D

added a method to specify hls with hlstime and removed hls_time option from Hls method to use default setting
@AndreaNicola
Copy link
Contributor Author

AndreaNicola commented Jul 12, 2024

@hixi Hi! i modified the method to use default hls_time and added a new method to set hls custom time :-)

Copy link

@hixi hixi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you :-)

@mowshon what do you think?

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.

2 participants