On Thu May 09, 2019 at 08:38:00PM +0200, Paco Esteban wrote:
> Hi ports@,
> 
> I sent this email to Brad Smith, maintainer of graphics/ffmpeg 2 weeks
> ago but did not have any answer.
> 
> I'm forwarding it here so maybe somebody can take a look.
> 
> Thanks.
> 
> ----- Forwarded message -----
> 
> ...
> 
> I had a problem the other day trying to watch a video stream using mpv
> (and later tried ffplay). I discovered that the OpenBSD package for
> ffmpeg does not support DASH demuxing, and that was the problem.
> 
> I prepared a diff to add that support, find it at the end of the email.
> 
> Just one thing, I'm not so experienced with ports (just built one and
> was really easy). I have a message like this when creating the package:
> 
>   LIB_DEPENDS textproc/libxml not needed for graphics/ffmpeg ?
> 
> I added that dependency myself, because DASH support is enabled via
> --enable-libxml2 at configure time ...
> Am I doing something wrong ?
> 

I like the improvement and it works for me. New diff with to two tweaks:

- Regen WANTLIB
- Minor bump libavformat because new symbols have been added.

RS

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/ffmpeg/Makefile,v
retrieving revision 1.182
diff -u -p -u -p -r1.182 Makefile
--- Makefile    2 Apr 2019 18:42:11 -0000       1.182
+++ Makefile    10 May 2019 05:06:24 -0000
@@ -8,11 +8,12 @@ CATEGORIES=   graphics multimedia
 MASTER_SITES=  https://ffmpeg.org/releases/
 EXTRACT_SUFX=  .tar.xz
 EPOCH=         0
+REVISION=      0
 
 SHARED_LIBS=   avcodec         23.0 \
                avdevice        11.0 \
                avfilter        8.0 \
-               avformat        20.0 \
+               avformat        20.1 \
                avresample      2.0 \
                avutil          13.0 \
                postproc        17.0 \
@@ -30,7 +31,7 @@ PERMIT_PACKAGE_FTP=   Yes
 WANTLIB=       SDL2 X11 Xext Xv ass bz2 c crypto dav1d>=2.0 fontconfig 
freetype \
                fribidi gsm m mp3lame opus pthread sndio speex ssl v4l2 \
                vorbis vorbisenc vpx x264 x265 xcb xcb-shape xcb-shm \
-               xcb-xfixes xvidcore z
+               xcb-xfixes xml2 xvidcore z
 
 BUILD_DEPENDS= textproc/texi2html \
                multimedia/frei0r-plugins
@@ -38,11 +39,11 @@ BUILD_DEPENDS=      textproc/texi2html \
 BUILD_DEPENDS+=        devel/nasm
 .endif
 LIB_DEPENDS=   archivers/bzip2 \
+               audio/gsm \
                audio/lame \
                audio/libvorbis \
                audio/opus \
                audio/speex \
-               audio/gsm \
                devel/sdl2 \
                multimedia/dav1d>=0.2.1 \
                multimedia/libass \
@@ -50,7 +51,8 @@ LIB_DEPENDS=  archivers/bzip2 \
                multimedia/libvpx \
                multimedia/x264 \
                multimedia/x265 \
-               multimedia/xvidcore
+               multimedia/xvidcore \
+               textproc/libxml
 
 # inter-library dependencies for the current configuration
 LIBavcodec_EXTRALIBS=-L${LOCALBASE}/lib -ldav1d -lgsm -lmp3lame -lopus -lspeex 
-lvorbis -lvorbisenc -lvpx -lx264 -lx265 -lxvidcore -lm -lz -pthread
@@ -110,6 +112,7 @@ CONFIGURE_ARGS+= --enable-shared \
                --enable-libx265 \
                --enable-libxvid \
                --enable-nonfree \
+               --enable-libxml2 \
                --enable-openssl \
                --extra-cflags="-I${LOCALBASE}/include -I${X11BASE}/include" \
                --extra-libs="-L${LOCALBASE}/lib -L${X11BASE}/lib" \

Reply via email to