On 06/20/11 10:12, Jasper Lievisse Adriaanse wrote:
> On Sun, Jun 19, 2011 at 11:17:28AM +0100, Nigel Taylor wrote:
>> On 06/19/11 07:52, Jacob Meuser wrote:
>>> On Sun, Jun 19, 2011 at 05:56:35AM +0100, Nigel Taylor wrote:
>>>
>>>> Goes wrong in configure, as diff shows. It's the pkg-config for avformat
>>>> that doesn't include avcodec, and avutil which causes the
>>>> av_register_all check to fail.
>>>
>>> ah. so it indeed is a change in Requires.private handling in
>>> pkg-config. or is it wrong that avutil and avcodec are in
>>> Requires.private instead of Requires?
>>>
>>> actually, -lavcodec -lavutil should be in libavformat's link list
>>> even without Requires. that was the intention with LIBavformat_EXTRALIBS
>>> in the port's MAKE_FLAGS and patches/patch-configure.
> The problem has always been there, it's just been uncovered by pkg-config
> that's more correct/strict now. It also fails with the freedesktop.org
> pkg-config.
>
>> ffmpeg - The libavformat.pc is created in the configure stage not build,
>> the MAKE_FLAGS has no effect. The
>> LIBavformat_EXTRALIBS=-L${LOCALBASE}/lib -lbz2 -lm
>> -Wl,-rpath,${LOCALBASE}/lib is used in creating the libavformat.pc, and
>> that doesn't contain the -lavcodec -lavutil that was added for the
>> MAKE_FLAGS.
> Right.
>
>> Regards
>>
>> Nigel Taylor
>
Hi,
The attached patch will allow transcode to build with the current
pkg-config. Builds on amd64 current.
Regards
Nigel Taylor
Index: Makefile
===================================================================
RCS file: /home/cvs/ports/multimedia/transcode/Makefile,v
retrieving revision 1.43
diff -u -p -r1.43 Makefile
--- Makefile 15 Jun 2011 23:16:15 -0000 1.43
+++ Makefile 19 Jun 2011 17:16:50 -0000
@@ -5,7 +5,7 @@ SHARED_ONLY= Yes
COMMENT= video stream processing tools
DISTNAME= transcode-1.1.5
-REVISION= 9
+REVISION= 10
CATEGORIES= multimedia
MASTER_SITES= ${MASTER_SITE_BERLIOS:=tcforge/}
EXTRACT_SUFX= .tar.bz2
@@ -98,7 +98,8 @@ CONFIGURE_ARGS+=--enable-a52 \
--enable-x264
CONFIGURE_ENV+= CPPFLAGS="-I${X11BASE}/include -I${LOCALBASE}/include" \
LDFLAGS="-L${X11BASE}/lib -L${LOCALBASE}/lib" \
- LIBAVCODEC_EXTRA_LIBS="-pthread"
+ LIBAVCODEC_EXTRA_LIBS="-pthread" \
+ LIBAVFORMAT_EXTRA_LIBS="-pthread -lavcodec"
.if ${FLAVOR:L:Mlzo}
CONFIGURE_ARGS+=--enable-lzo
