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