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

perf: rtsp: replace gst-rtsp-server with mediamtx #577

Merged
merged 2 commits into from
Nov 23, 2024

Conversation

CodeChenL
Copy link
Contributor

Description of changes

@CodeChenL CodeChenL requested a review from a team as a code owner November 20, 2024 02:46
Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

所以为什么gst-rtsp-server不能用了呢?

@CodeChenL
Copy link
Contributor Author

所以为什么gst-rtsp-server不能用了呢?

很久没人维护这个库了,导致debian更新gst版本后,它的编译脚本找不到gstreamer-rtsp的位置了

@RadxaYuntian
Copy link
Member

你现在用的版本也不是最新的呀,最新的gst-rtsp-server也有同样的问题么?

@CodeChenL
Copy link
Contributor Author

CodeChenL commented Nov 20, 2024

你现在用的版本也不是最新的呀,最新的gst-rtsp-server也有同样的问题么?

试过github上的最新代码不行,刚刚发现gitlab.freedesktop.org上有更新https://gitlab.freedesktop.org/gstreamer/gstreamer/-/tree/main/subprojects/gst-rtsp-server

@RadxaYuntian
Copy link
Member

如果gitlab上的可以用的话可以两个都保留,直到rtsp-server不能用为止

@CodeChenL
Copy link
Contributor Author

meson给的信息太少了,自己写了个cmake导入gst,报错找不到librga。
发现是因为rkr4的gst版本高于debian源的gst,优先安装sdk的gst。
然后sdk的gst的pkgconfig文件相比debian多了librga,所以导致为什么以前可以找到gst-rtsp依赖现在不行。
通过安装librga-dev解决,而且比以前的推流效果好很多。

@CodeChenL
Copy link
Contributor Author

先提中文

Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

整理一下提交

@peterwang2050 peterwang2050 merged commit d96d6e7 into radxa-docs:main Nov 23, 2024
6 checks passed
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