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

Avoid libv4l2 pkg-config dependency added by mistake #106

Merged
merged 1 commit into from
Sep 1, 2024

Conversation

vladmovchan
Copy link
Contributor

@vladmovchan vladmovchan commented Aug 12, 2024

User API header file required to generate bindings in v4l2-sys is not part of libv4l2.

Instead, directly check if UAPI header file is present on FreeBSD and extend compiler's include paths.
Or guide user towards necessary package if header file is missing.

@vladmovchan vladmovchan mentioned this pull request Aug 12, 2024
@MarijnS95
Copy link
Collaborator

Don't forget to add a comment to the code explaining that we don't need the libv4l2 headers either to build v4l2-sys, just the Linux UAPI header.

I'm not certain whether that's provided by libv4l2 or another package on FreeBSD. Is it https://www.freshports.org/multimedia/v4l_compat?

v4l2-sys/Cargo.toml Outdated Show resolved Hide resolved
v4l2-sys/build.rs Outdated Show resolved Hide resolved
v4l2-sys/build.rs Outdated Show resolved Hide resolved
@vladmovchan vladmovchan changed the title Limit pkg-config use to FreeBSD only Avoid libv4l2 pkg-config dependency added by mistake Aug 12, 2024
v4l2-sys/build.rs Outdated Show resolved Hide resolved
User API header file required to generate bindings in `v4l2-sys` is not
part of `libv4l2`.

Instead directly check if UAPI header file is present on FreeBSD and
extend compiler's include paths. Or guide user towards necessary package
if header file is missing.
@raymanfx raymanfx merged commit 677b02d into raymanfx:master Sep 1, 2024
5 checks passed
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.

3 participants