Created attachment 154796 [details]
mplayer2 build log
I reinstalled all ports related to ogg/theora, but it fails to link
-lXinerama -lXxf86vm -lSDL -lGL
libmpdemux/demux_ogg.o: In function `demux_ogg_read_packet':
libmpdemux/demux_ogg.c:(.text+0x4319): undefined reference to `_ilog'
cc: error: linker command failed with exit code 1 (use -v to see invocation)
Makefile:549: fallo en las instrucciones para el objetivo 'mplayer'
gmake: *** [mplayer] Error 1
gmake: se sale del directorio '/usr/ports/multimedia/mplayer2/work/mplayer2-2.0.20130428'
===> Compilation failed unexpectedly.
Try to set MAKE_JOBS_UNSAFE=yes and rebuild before reporting the failure to
*** Error code 1
Assign to maintainer.
Created attachment 155022 [details]
Created attachment 155023 [details]
The following patches solves my problem.
Do you have a poudriere build log ?
(In reply to chris from comment #4)
Building on CURRENT...
Created attachment 155653 [details]
mplayer2 build log on 11-CURRENT
Here is my configuration options:
% make pretty-print-config
+ASS -CACA -DEBUG -DV -GIF +IPV6 -JACK -LADSPA -LIBBLURAY +LIBCDIO -LIRC +MAD +OPENGL -PORTAUDIO -PULSE -REALPLAYER -RTC +SDL -SMB +SPEEX +THEORA -VDPAU +WIN32 +X11 +XINERAMA
How can you say this patch fixes it for me if it fails to build ?
Could you show the error(s)? It's weird :S
YOu first send in a PR regarding theora, then you send in 2 patches saying those patches fixes the issue for you, I request a build log and it shows the build fails so I don't know how those patches "fixes the issue" if the build fails.
(In reply to chris from comment #10)
No. It builds fine, see that attachment:
Ah I was looking at the "mplayer2 build log" not the other one, bad day to stop taking my meds. I will test this on my end and either Approve it or deny it. Thank you for your patches.
Hehe, it don't matter. Go ahead :)
(In reply to Carlos Jacobo Puga Medina from comment #13)
So is it both patches or one or the other ?
(In reply to chris from comment #15)
You need both patches.
OK thank you sir, I will test in a bit after I free my mind from looking at C all day.
Reset assignee per ports r384126 and ports r384127.
Build tested on 10.1a, 9.3a, 8.4i, looks fine.
A commit references this bug:
Date: Thu Apr 30 06:45:27 UTC 2015
New revision: 385026
multimedia/mplayer2: fix build if THEORA support enabled
Submitted by: Carlos Jacobo Puga Medina <firstname.lastname@example.org>
Approved by: email@example.com (maintainer)