[FFmpeg-devel] [PATCH] fate: Add EXIF test.

[Michael Niedermayer]

On Wed, Aug 21, 2013 at 10:21:02AM +0200, Thilo Borgmann wrote:
> Am 21.08.13 00:08, schrieb Paul B Mahol:
> > On 8/20/13, Thilo Borgmann <thilo.borgmann at mail.de> wrote:
> >> Am 20.08.13 22:44, schrieb Michael Niedermayer:
> >>> On Tue, Aug 20, 2013 at 06:57:12PM +0200, Michael Niedermayer wrote:
> >>>> On Tue, Aug 20, 2013 at 04:00:19PM +0200, Thilo Borgmann wrote:
> >>>>> Rev 2 attached.
> >>>>>
> >>>>> Please check if the dependencies for the checks on TIFF, JPEG & MP3 make
> >>>>> sense
> >>>>> now...
> >>>>
> >>>> applied
> >>>>
> >>>> thanks
> >>>
> >>> seems this breaks
> >>> http://fate.ffmpeg.org/report.cgi?time=20130820145035&slot=x86_64-archlinux-gcc-random
> >>
> >> from there:
> >>> LD	ffprobe_g
> >>> libavcodec/libavcodec.a(mjpegdec.o): In function `mjpeg_decode_app':
> >>> /home/ux/fate/ffmpeg/libavcodec/mjpegdec.c:1523: undefined reference to
> >>> `ff_exif_decode_ifd'
> >>> collect2: error: ld returned 1 exit status
> >>> make: *** [ffserver_g] Error 1
> >>
> >> this should have been broken since commit
> >> bb4e1b4cf910af0de2bc884c75544603c40010cc
> >>
> >> Since it works for me on my machines, I can only guess what is wrong :(
> > 
> > Makefile needs updating.
> 
> So my guess is that this will fix it because exif.o has not been build according
> to the log file.

fixes it but i think its better to have exif as a component in
configure instead of listing all exif files for all decoders that use
it.
(that is like for exmple golomb is handled)
[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130821/bf265ffb/attachment.asc>