Conversation
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/6156 |
| @@ -112,6 +112,7 @@ stdenv.mkDerivation rec { | |||
| "-Dv4l2=true" | |||
| (lib.mesonEnable "tracing" withTracing) | |||
| (lib.mesonEnable "qcam" withQcam) | |||
| "-Dlibunwind=disabled" | |||
There was a problem hiding this comment.
Is there a reason we do not want this? If so - that should be in a comment.
There was a problem hiding this comment.
This is what we were effectively doing before, the option was just made explicit now so we have to put this here instead of just compiling without libunwind.
Libunwind has been disabled the entire time we've been packaging libcamera, and I don't see any explanation of why that is, so I'd rather not comment with an uneducated guess here.
There was a problem hiding this comment.
libunwind is for debugging, so it seems reasonable to disable it. (I'd have used lib.mesonEnable but it's whatever.) Arguably it could depend on withTracing.
|
(CCing maintainers who have been involved in previous libcamera bumps. Sorry for the noise.) |
|
Successfully created backport PR for |
Changelog: https://git.libcamera.org/libcamera/libcamera.git/tag/?h=v0.6.0
Release notes: https://lists.libcamera.org/pipermail/libcamera-devel/2025-November/055244.html
I'm hoping that this will help with #225743
Hans de Goede has pushed a number of fixes to components such as ipa_soft_simple that have been problematic for IPU6 cameras, which will be included in this version.
I've built all the direct dependers, and they seem to be functioning OK.
I'm also rebuilding my system to test this out more thoroughly, but it might take a while due to the number of rebuilds here.Update: I have rebuilt my system with no regressions, and confirmed that this fixes my webcam issues.Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.