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

Update camera-streamer.sh #221

Closed
wants to merge 1 commit into from
Closed

Update camera-streamer.sh #221

wants to merge 1 commit into from

Conversation

imserhii
Copy link

Problem: can't open 8080 outside machine
Fix: same behaviour as in ustreamer

Problem: can't open 8080 outside machine
Fix: same behaviour as in ustreamer
@mryel00
Copy link
Member

mryel00 commented Dec 19, 2023

Hey, first of all thanks for the PR 😄
But now to the problem why I didn't add this myself yet. Did you test this on bullseye? I don't think so. The problem is that we currently are using two different camera-streamer versions. One that got this parameter on bookworm and one without that parameter on bullseye (and sometimes on bookworm).
Now you might think. Just check for bookworm or bullseye then, but we will run into more problems with that. You see, I updated the make update command recently. That will update crowsnest and recompiles ustreamer and camera-streamer. It's doing that by recloning camera-streamer, we got our own reasons for that. One reason can be seen inside #216. That PR is currently a complete mess atm and will be rewritten soon.

But I oversaw one thing doing that change to make update. Currently all distributions download master of camera-streamer.
So now we got people with bullseye on master, those will get problems with that parameter and camera-streamer won't start anymore.
Then we got the bookworm people on main, those will profit from that change.
The last party are people with bookworm on master, as they used make update recently to update and recompile camera-streamer, since there was an update to libcamera. For that they needed an update for camera-streamer and recompile it.

So now we have to run a git branch check to correctly identify the systems to see if that parameter might break something. That's why I didn't yet add that parameter myself, as it's not that hard to just add that one parameter yourself.

With all that you now know on what you should take into account to adjust this PR before I can merge it 😅

P.S. Please use conventional commits as you can also read on the Contribute chapter. This is most important for the PR title

@mryel00
Copy link
Member

mryel00 commented Jan 28, 2024

Closing this due to no further interaction and the implementation with #238

@mryel00 mryel00 closed this Jan 28, 2024
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