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

Added support for RTSP #115

Closed
wants to merge 1 commit into from
Closed

Added support for RTSP #115

wants to merge 1 commit into from

Conversation

bartekpacia
Copy link

I'm making a PR for changes made by @e1z0 in his issue.

It works really great

Co-authored-by: e1z0 github.com/e1z0
@bartekpacia
Copy link
Author

please @brutella, review this

@bartekpacia
Copy link
Author

: (

@brutella
Copy link
Owner

brutella commented Mar 1, 2022

Sorry for not looking into this so far. Your pull request results in a lot of duplicated code. It would be great if we could fix that.

Could you please summarise which FFmpeg options have to be changed to make RTSP work. Thanks.

@tomtastic
Copy link

Hi @bartekpacia, will you have time to resolve these questions, I'd love to see your work merged here :)

@bartekpacia
Copy link
Author

Hahah, I totally forgot about this PR. Will look today into it.

@bartekpacia
Copy link
Author

bartekpacia commented Nov 6, 2022

Actually what I did in this PR is only apply a patch from @e1z0 that he shared in this issue. I didn't write this code myself. Also, I see that some merge conflicts arose.

@tomtastic Feel free to pick up this PR and create a new one.

@bartekpacia bartekpacia closed this Nov 6, 2022
@bartekpacia bartekpacia deleted the add_rtsp branch November 4, 2023 16:48
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.

3 participants