Hi,

I'd like to suggest the attached patch as a starting point. It gets PV compiled
again against ffmpeg svn head. On 32 and 64 bit Linux and 64 bit Solaris systems
I did not observe any side effects yet (using gcc only to compile PV, though).

Sven



Dave Partyka wrote, On 29.04.2010 17:46:
> FYI, It is on my todo list to
> 
> 1. restore dash16 to build VTK against the old FFMPEG
> 2. potentially setup a new dashboard that builds against at least a more
> recent version.
> 
> On Thu, Apr 29, 2010 at 11:38 AM, Sven Buijssen
> <[email protected] <mailto:[email protected]>> wrote:
> 
>     Michael,
> 
>     > The problem with ffmpeg IMHO is that it is a moving target (as is
>     also declared on their website). It is almost impossible for
>     ParaView to keep up and just arbitrarily picking any revision is no
>     solution either, because that will cause a lot of trouble for packagers.
> 
>     It's just that PV and VTK for a month or three now supported both
>     the old
>     outdated ffmpeg version from February 2007 or so (downloadable as a
>     tarball from
>     the addons section of the VTK software download website) AND the
>     latest ffmpeg.
>     Up to last Monday. Given the effort spent to re-add support for
>     newer ffmpeg
>     versions and considering that the change in ffmpeg that breaks it
>     this time in
>     PV is less complex to work around than last time (headers completely
>     reworked),
>     I think we shouldn't let go that easily.
> 
>     > <shameless_plug>I suggest you give the included Ogg/Theora writer
>     a try.</shameless_plug>.
> 
>     I'm well aware of your contribution and compile ParaView with
>      -DVTK_USE_OGGTHEORA_ENCODER:BOOL=ON -DVTK_USE_FFMPEG_ENCODER:BOOL=ON
>     ever since you announced your git fork of PV on this mailing list,
>     so even
>     before ogv support has been added to the official source tree. :-)
>      It's just
>     that some of my users prefer avi over ogv.
> 
>     Sven
>     _______________________________________________
>     Powered by www.kitware.com <http://www.kitware.com>
> 
>     Visit other Kitware open-source projects at
>     http://www.kitware.com/opensource/opensource.html
> 
>     Please keep messages on-topic and check the ParaView Wiki at:
>     http://paraview.org/Wiki/ParaView
> 
>     Follow this link to subscribe/unsubscribe:
>     http://www.paraview.org/mailman/listinfo/paraview
> 
> 

>From 06c8e3e38f5fec2fe0d300ff7f7ab823a03952e3 Mon Sep 17 00:00:00 2001
From: Sven Buijssen <[email protected]>
Date: Thu, 29 Apr 2010 18:29:20 +0200
Subject: [PATCH] COMP: Fix for ffmpeg revision 22965 and newer

---
 IO/CMakeLists.txt |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/IO/CMakeLists.txt b/IO/CMakeLists.txt
index 72cee22..a0a068f 100644
--- a/IO/CMakeLists.txt
+++ b/IO/CMakeLists.txt
@@ -292,6 +292,10 @@ IF (VTK_USE_FFMPEG_ENCODER)
     SET(KIT_LIBS ${KIT_LIBS}
       ${FFMPEG_swscale_LIBRARY})
   ENDIF (NOT VTK_FFMPEG_HAS_IMG_CONVERT)
+
+  IF(NOT VTK_FFMPEG_HAS_OLD_HEADER)
+    ADD_DEFINITIONS(-D__STDC_CONSTANT_MACROS)
+  ENDIF(NOT VTK_FFMPEG_HAS_OLD_HEADER)
 ENDIF (VTK_USE_FFMPEG_ENCODER)
 
 #-----------------------------------------------------------------------------
-- 
1.7.1

_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the ParaView Wiki at: 
http://paraview.org/Wiki/ParaView

Follow this link to subscribe/unsubscribe:
http://www.paraview.org/mailman/listinfo/paraview

Reply via email to