Skip to content

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