Skip to content

v4l-utils: make BPF support optional#467296

Merged
nixpkgs-ci[bot] merged 1 commit intoNixOS:stagingfrom
dotlambda:v4l-utils
Dec 3, 2025
Merged

v4l-utils: make BPF support optional#467296
nixpkgs-ci[bot] merged 1 commit intoNixOS:stagingfrom
dotlambda:v4l-utils

Conversation

@dotlambda
Copy link
Member

see #419046 (review)

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". label Dec 2, 2025
@nixpkgs-ci nixpkgs-ci bot requested a review from codyopel December 2, 2025 22:43
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Dec 2, 2025
Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

LGTM

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Dec 2, 2025
(lib.mesonOption "gconvsysdir" "${glibc.out}/lib/gconv")
]
++ lib.optionals (stdenv.hostPlatform != stdenv.buildPlatform) [
# BPF support fail to cross compile, unable to find `linux/lirc.h`
Copy link
Member Author

Choose a reason for hiding this comment

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

@avnik I can't reproduce this failure using pkgsCross.aarch64-multiplatform.v4l-utils on x86_64-linux. See #429900 for the PR that made this conditional on non-cross.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm re-enabling BPF support in #467632.

Copy link
Contributor

@Yarny0 Yarny0 left a comment

Choose a reason for hiding this comment

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

I tested on x86_64-linux, with withBPF=true and withBPF=false, after cherry-picking onto current nixos-unstable:

  • The output of v4l2-ctl --list-formats-ext -d /dev/video${NUMBER} seems alright.
  • dvbv5-zap --adapter=${ADAPTER} --channel=channels.dvbv5 --pat --output=/tmp/output.ts ${CHANNEL_NAME} records some TV into the output file as expected (tested with dvb-t2 and dvb-s2).
  • qv4l2 starts up and looks nice (I haven't tested any hardware access with this program).
  • dtv-scan-tables builds (it uses ${v4l-utils}/bin/dvb-format-convert during the build).
  • dvbv5-scan --get_frontend --nit ${dtv-scan-tables}/share/dvbv5/dvb-t/${SOME_TRANSPONDER_FILE} finds a lot of dvb-t channels, as it should.
  • mpv av://v4l2:/dev/video${NUMBER} shows a webcam's video stream.
  • mpv dvb://${CHANNEL} works (tested with dvb-s).

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Dec 3, 2025
@Yarny0
Copy link
Contributor

Yarny0 commented Dec 3, 2025

@NixOS/nixpkgs-merge-bot merge

@nixpkgs-ci nixpkgs-ci bot added this pull request to the merge queue Dec 3, 2025
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Dec 3, 2025

@Yarny0 wants to merge this PR.

Requirements to merge this PR with @NixOS/nixpkgs-merge-bot merge:

  • ✅ PR targets a development branch.
  • ✅ PR touches only files of packages in pkgs/by-name/.
  • ✅ PR is at least one of:
    • ✅ Approved by a committer.
    • ⬜ Backported via label.
    • ✅ Opened by a committer.
    • ⬜ Opened by r-ryantm.
  • ✅ Yarny0 is a member of @NixOS/nixpkgs-maintainers.
  • ✅ Yarny0 is a maintainer of all touched packages.

✔️ Queued for merge (#306934)

Merged via the queue into NixOS:staging with commit a607eed Dec 3, 2025
36 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants