Ok. 

—
Antoine

> On 29 Oct 2023, at 14:43, Landry Breuil <[email protected]> wrote:
> 
> Le Sun, Oct 29, 2023 at 02:32:40PM +0100, Landry Breuil a écrit :
>> Le Sun, Oct 29, 2023 at 02:23:59PM +0100, Antoine Jacoutot a écrit :
>>> Any reason we don’t want support for it ?
>> 
>> in our case its just to avoid build breakage :)
>> 
>>> I mean the whole point in hooking it up is to be able to use it, no?
>> 
>> im precisely looking at libv4l, and the problem is that if libudev is
>> found it tries to build libdvbv5 which fails to build because of a
>> missing linux-only header.
> 
> if libudev is found, the build fails here:
> 
> In file included from 
> /usr/obj/ports/libv4l-1.20.0/v4l-utils-1.20.0/lib/libdvbv5/compat-soname.c:21:
>                       
> In file included from 
> /usr/obj/ports/libv4l-1.20.0/v4l-utils-1.20.0/lib/include/libdvbv5/dvb-fe.h:31:
>                      
> /usr/obj/ports/libv4l-1.20.0/v4l-utils-1.20.0/lib/include/libdvbv5/dvb-frontend.h:30:10:
>  fatal error: 'linux/types.h' file not found
> #include <linux/types.h>
> 
>> 
>> so with libinput/libudev installed, libv4l builds with
>> --disable-libdvbv5, without needing to patch configure goo, but
>> unexpected things might happen if in a future libv4l upgrade more bits
>> are enabled if libudev is found.
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/multimedia/libv4l/Makefile,v
> retrieving revision 1.32
> diff -u -r1.32 Makefile
> --- Makefile    27 Oct 2023 15:23:45 -0000      1.32
> +++ Makefile    29 Oct 2023 13:42:49 -0000
> @@ -30,6 +30,7 @@
> LIBTOOL_FLAGS =                --tag=disable-static
> CONFIGURE_STYLE =      autoconf
> CONFIGURE_ARGS =       --disable-doxygen-doc \
> +                       --disable-libdvbv5 \
>                        --disable-static
> CONFIGURE_ENV =                CPPFLAGS="${CPPFLAGS}" \
>                        LDFLAGS="-L${LOCALBASE}/lib"
> 
> ok for this version ?

Reply via email to