Hi David,

On Thu, Mar 12, 2009 at 6:45 PM, David Callu <[email protected]> wrote:
>         FIND_PATH(FFMPEG_${varname}_INCLUDE_DIRS ${headername}
>             ${FFMPEG_ROOT}/include
>             $ENV{FFMPEG_DIR}/include
>             $ENV{OSGDIR}/include
>             $ENV{OSG_ROOT}/include
>             ~/Library/Frameworks
>             /Library/Frameworks
>             /usr/local/include
>             /usr/include/
>             /usr/include/ffmpeg  #                         <<-- this line
>             /sw/include # Fink
>             /opt/local/include # DarwinPorts
>             /opt/csw/include # Blastwave
>             /opt/include
>             /usr/freeware/include
>         )

This seems like a sensible change.

Do you want to revise you changes then resubmit?

Something else I've considered is simplying the ffmpeg/CMakeLists.txt
so it just includes the directories directly without adding any extra
paths, this pushes the FindFFmpeg.cmake do to a thorough job, but that
is the normal practice for our Find scripts.

Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to