Hi Ingo, I was about to commit this when I noticed that you didn't add a Signed-off-by line in your email. We need that for v4l-utils. See section 11 here: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=HEAD for details about that tag. Just reply with your Signed-off-by and I'll merge this patch. Regards, Hans On 03/13/2017 12:58 PM, Ingo Feinerer wrote: > Hi, > > please find attached a diff that makes the inclusion of the sys/sysmacros.h > header file conditional. I noticed it on OpenBSD which has no sys/sysmacros.h, > so compilation fails there. > > Best regards, > Ingo > > diff --git a/configure.ac b/configure.ac > index f3269728a..ae58da377 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -146,6 +146,7 @@ if test "x$gl_cv_func_ioctl_posix_signature" = xyes; then > fi > > AC_CHECK_FUNCS([__secure_getenv secure_getenv]) > +AC_HEADER_MAJOR > > # Check host os > case "$host_os" in > diff --git a/lib/libv4lconvert/control/libv4lcontrol.c b/lib/libv4lconvert/control/libv4lcontrol.c > index 59f28b137..1e784eda8 100644 > --- a/lib/libv4lconvert/control/libv4lcontrol.c > +++ b/lib/libv4lconvert/control/libv4lcontrol.c > @@ -20,7 +20,9 @@ > */ > > #include <sys/types.h> > +#if defined(MAJOR_IN_SYSMACROS) > #include <sys/sysmacros.h> > +#endif > #include <sys/mman.h> > #include <fcntl.h> > #include <sys/stat.h> >
