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

V4L2rtsp streaming server: new recipe #774

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mandraga
Copy link

I closed the previous pull request because it was on master and it became a mess if I wanted to squash the commits into one.
This pull request adds a recipe for v4l2rtserver.
A year ago, the repo was worked on, such as it compiles neatly.

Patrick Areny and others added 4 commits September 15, 2022 17:07
    v4l2rtspserver is an streamer feed from :
      - Video4Linux devices that support H264, HEVC, JPEG, VP8 or VP9 capture.
      - ALSA devices that support PCM S16_BE, S16_LE, S32_BE or S32_LE

Signed-off-by: Patrick Areny <patrick.areny@notiloplus.com>
The server has a tagged 0.3.3 release on an earlier commit. Only the latest commits works so we specify this on the PV variable.
We are somewehre between the 0.3.3 release and the future 0.3.4.

Signed-off-by: Patrick Areny <patrick.areny@notiloplus.com>
v4l2srtpserver: remove useless PV value
@mandraga mandraga force-pushed the addrecipev4l2rtserver branch from bb814a4 to fa290b9 Compare January 16, 2024 22:47
@mandraga
Copy link
Author

I removed the PV statement.
I squashed the commits, is this necessary? Seems dirty to me given the squash commit option from the reviewer.

@kraj
Copy link
Contributor

kraj commented Jan 16, 2024

I removed the PV statement. I squashed the commits, is this necessary? Seems dirty to me given the squash commit option from the reviewer.

Please squash and rebase all changes into single commit and push to the pull branch. We want to keep history clean

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