Hi Jim,

I've just changed the XRANDR_INCLUDE_DIRS so that they get added to
the LIB_PRIVATE_HEADERS rather than the LIB_PUBLIC_HEADERS, my build
under Linux still works fine, could you try it out on your system.
The change is now checked into svn/trunk.
src/osgViewer/CMakeLists.txt is also attached.

Robert.

On Sun, Jan 25, 2009 at 8:46 PM, Robert Osfield
<[email protected]> wrote:
> Hi Jim,
>
> Thanks for the extra feedback.  Could you post the values that
> PKG_CONFIG aquires for XRANDR?
>
> Given the install uses LIB_PUBLIC_HEADERS, I wonder if the following
> line in the XRANDR setup code in src/osgViewer/CMakeLists.txt is the
> source of the problem:
>
>            SET(LIB_PUBLIC_HEADERS ${LIB_PUBLIC_HEADERS} 
> ${XRANDR_INCLUDE_DIRS})
>
> Perhaps moving the XRANDR_INCLUDE_DIRS to ADD_LIBRARY entry rather
> than LIB_PUBLIC_HEADERS would work around this.
>
> Robert.
>
> On Sun, Jan 25, 2009 at 8:12 PM, Jim Brooks <[email protected]> wrote:
>>>I've just reviewed your change and feel that it perhaps isn't solving
>>>the problem, rather just hacking around it.
>>>
>>>Which version of CMake are you using?
>>
>>
>> Haven't uncovered the real problem yet, sorry.
>>
>> cmake 2.6.1 (and tried 2.6.2)
>>
>>
>>>The original code block that you are disabling shouldn't produce
>>>errors, as checks are in place to not enable the XRANDR support if
>>>PKG_COFNIG is not supported or that XRANDR is not available.  The code
>>>block is:
>>
>>
>> If this line is commented-out, then cmake will succeed:
>>
>>  # PKG_CHECK_MODULES(XRANDR xrandr)
>>
>>
>>>Is it that FreeBSD does support XRANDR?
>>
>>
>> Yes, FreeBSD has XRANDR.
>>
>>
>>> Is it an issue with the LIB_PUBLIC_HEADERS?
>>
>>
>> Don't think so. That seems to be defined ok.
>>
>> Rather, if PKG_CHECK_MODULES() isn't commented-out,
>> it breaks INCLUDE() and INSTALL() in the last section (don't know why):
>>
>> ...............................................
>> INCLUDE(ModuleInstall OPTIONAL)
>>
>> FOREACH( INCLUDEFILE ${LIB_PUBLIC_HEADERS} )
>>   FILE(RELATIVE_PATH REL_INCLUDEFILE ${HEADER_PATH} ${INCLUDEFILE})
>>   GET_FILENAME_COMPONENT(REL_INCLUDE_PATH ${REL_INCLUDEFILE} PATH)
>>   INSTALL(
>>   FILES        ${INCLUDEFILE}
>>   DESTINATION ${INSTALL_INCDIR}/${LIB_NAME}/${REL_INCLUDE_PATH}
>>   COMPONENT libopenscenegraph-dev
>>   )
>> ENDFOREACH( INCLUDEFILE)
>> ...............................................
>> _______________________________________________
>> osg-submissions mailing list
>> [email protected]
>> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>>
>
# FIXME: For OS X, need flag for Framework or dylib
IF   (DYNAMIC_OPENSCENEGRAPH)
    ADD_DEFINITIONS(-DOSGVIEWER_LIBRARY)
ELSE (DYNAMIC_OPENSCENEGRAPH)
    ADD_DEFINITIONS(-DOSG_LIBRARY_STATIC)
ENDIF(DYNAMIC_OPENSCENEGRAPH)

SET(LIB_NAME osgViewer)

SET(HEADER_PATH ${OpenSceneGraph_SOURCE_DIR}/include/${LIB_NAME})
SET(LIB_PUBLIC_HEADERS
    ${HEADER_PATH}/CompositeViewer
    ${HEADER_PATH}/Export
    ${HEADER_PATH}/GraphicsWindow
    ${HEADER_PATH}/Renderer
    ${HEADER_PATH}/Scene
    ${HEADER_PATH}/Version
    ${HEADER_PATH}/View
    ${HEADER_PATH}/Viewer
    ${HEADER_PATH}/ViewerBase
    ${HEADER_PATH}/ViewerEventHandlers
)

SET(LIB_COMMON_FILES
    CompositeViewer.cpp
    HelpHandler.cpp
    Renderer.cpp
    Scene.cpp
    ScreenCaptureHandler.cpp
    StatsHandler.cpp
    Version.cpp
    View.cpp
    Viewer.cpp
    ViewerBase.cpp
    ViewerEventHandlers.cpp
)

SET(LIB_EXTRA_LIBS)

IF(WIN32)

    #
    # Enable workaround for OpenGL driver issues when used in 
multithreaded/multiscreen with NVidia drivers on Windows XP 
    # For example: osgviewer dumptruck.osg was showing total garbage (screen 
looked like shattered, splashed hedgehog) 
    # There were also serious issues with render to texture cameras.
    # Workaround repeats makeCurrentContext call as it was found that this 
causes the problems to dissapear.
    #
    OPTION(OSG_MULTIMONITOR_MULTITHREAD_WIN32_NVIDIA_WORKAROUND "Set to ON if 
you have NVidia board and drivers earlier than 177.92 ver" OFF)
    MARK_AS_ADVANCED(OSG_MULTIMONITOR_MULTITHREAD_WIN32_NVIDIA_WORKAROUND)
    IF(OSG_MULTIMONITOR_MULTITHREAD_WIN32_NVIDIA_WORKAROUND)
        ADD_DEFINITIONS(-DOSG_MULTIMONITOR_MULTITHREAD_WIN32_NVIDIA_WORKAROUND)
    ENDIF(OSG_MULTIMONITOR_MULTITHREAD_WIN32_NVIDIA_WORKAROUND)

    SET(LIB_PUBLIC_HEADERS ${LIB_PUBLIC_HEADERS}
        ${HEADER_PATH}/api/Win32/GraphicsWindowWin32
        ${HEADER_PATH}/api/Win32/PixelBufferWin32
    )
        
    SET(LIB_COMMON_FILES ${LIB_COMMON_FILES} 
        GraphicsWindowWin32.cpp
        PixelBufferWin32.cpp
    )

ELSE(WIN32)

    IF(APPLE)
        SET(OSG_WINDOWING_SYSTEM "Carbon" CACHE STRING "Windowing system type 
for graphics window creation, options Carbon or X11.")
    ELSE(APPLE)
        SET(OSG_WINDOWING_SYSTEM "X11" CACHE STRING "Windowing system type for 
graphics window creation. options only X11")
    ENDIF(APPLE)
    

    IF(${OSG_WINDOWING_SYSTEM} STREQUAL "Carbon")

        # FIXME: OS X needs selection mechanism for Cocoa, Carbon, X11
        SET(LIB_PUBLIC_HEADERS ${LIB_PUBLIC_HEADERS} 
            ${HEADER_PATH}/api/Carbon/GraphicsWindowCarbon
            ${HEADER_PATH}/api/Carbon/PixelBufferCarbon
        )
        SET(LIB_COMMON_FILES ${LIB_COMMON_FILES} 
            GraphicsWindowCarbon.cpp
            PixelBufferCarbon.cpp
        )

    ELSE(${OSG_WINDOWING_SYSTEM} STREQUAL "Carbon")

        # X11 for everybody else


        INCLUDE(FindPkgConfig OPTIONAL)
        IF (PKG_CONFIG_FOUND)

            PKG_CHECK_MODULES(XRANDR xrandr)
            
            IF (XRANDR_FOUND) 
                OPTION(OSGVIEWER_USE_XRANDR "Set to ON to enable Xrandr support 
for GraphicsWindowX11." ON)
            ELSE(XRANDR_FOUND) 
                SET(OSGVIEWER_USE_XRANDR OFF)
            ENDIF (XRANDR_FOUND)
        ELSE(PKG_CONFIG_FOUND)
            SET(OSGVIEWER_USE_XRANDR OFF)
        ENDIF(PKG_CONFIG_FOUND)


        SET(LIB_PUBLIC_HEADERS ${LIB_PUBLIC_HEADERS}
            ${HEADER_PATH}/api/X11/GraphicsWindowX11
            ${HEADER_PATH}/api/X11/PixelBufferX11
        )


        SET(LIB_COMMON_FILES ${LIB_COMMON_FILES} 
            GraphicsWindowX11.cpp 
            PixelBufferX11.cpp
        )
        
        IF(OSGVIEWER_USE_XRANDR)
            ADD_DEFINITIONS(-DOSGVIEWER_USE_XRANDR)
            SET(LIB_PRIVATE_HEADERS ${LIB_PRIVATE_HEADERS} 
${XRANDR_INCLUDE_DIRS} )
            LINK_LIBRARIES(Xrandr)
        ENDIF(OSGVIEWER_USE_XRANDR)

        # X11 on Apple requires X11 library plus OpenGL linking hack on Leopard
        IF(APPLE)
            # Find GL/glx.h 
            IF(EXISTS ${CMAKE_OSX_SYSROOT}/usr/X11/include/GL/glx.h)
                SET(OPENGL_INCLUDE_DIR /usr/X11/include)
                SET(OPENGL_LIBRARIES /usr/X11/lib/libGL.dylib)
            ELSEIF(EXISTS ${CMAKE_OSX_SYSROOT}/usr/X11R6/include/GL/glx.h)
                SET(OPENGL_INCLUDE_DIR /usr/X11R6/include)
                SET(OPENGL_LIBRARIES /usr/X11R6/lib/libGL.dylib)
            ENDIF(EXISTS ${CMAKE_OSX_SYSROOT}/usr/X11/include/GL/glx.h)
            INCLUDE_DIRECTORIES(BEFORE SYSTEM ${OPENGL_INCLUDE_DIR})

            SET(LIB_EXTRA_LIBS ${X11_X11_LIB} ${OPENGL_LIBRARIES} 
${LIB_EXTRA_LIBS})
            SET(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} 
-Wl,-dylib_file,/System/Library/Frameworks/OpenGL.framework/Versions/A/Libraries/libGL.dylib:${CMAKE_OSX_SYSROOT}/System/Library/Frameworks/OpenGL.framework/Versions/A/Libraries/libGL.dylib")
        ENDIF(APPLE)
            

    ENDIF(${OSG_WINDOWING_SYSTEM} STREQUAL "Carbon")

ENDIF(WIN32)


ADD_LIBRARY(${LIB_NAME}
    ${OPENSCENEGRAPH_USER_DEFINED_DYNAMIC_OR_STATIC}
    ${LIB_PUBLIC_HEADERS}
    ${LIB_PRIVATE_HEADERS}
    ${LIB_COMMON_FILES}
)


LINK_INTERNAL(${LIB_NAME}     
    osgGA
    osgText
    osgDB
    osgUtil
    osg
    OpenThreads
    ${LIB_EXTRA_LIBS}
)

LINK_CORELIB_DEFAULT(${LIB_NAME})

IF(MINGW OR CYGWIN)
    LINK_EXTERNAL(${LIB_NAME} gdi32 )
ENDIF(MINGW OR CYGWIN)

INCLUDE(ModuleInstall OPTIONAL)

FOREACH( INCLUDEFILE ${LIB_PUBLIC_HEADERS} )
   FILE(RELATIVE_PATH REL_INCLUDEFILE ${HEADER_PATH} ${INCLUDEFILE})
   GET_FILENAME_COMPONENT(REL_INCLUDE_PATH ${REL_INCLUDEFILE} PATH)
   INSTALL(
   FILES        ${INCLUDEFILE}
   DESTINATION ${INSTALL_INCDIR}/${LIB_NAME}/${REL_INCLUDE_PATH}
   COMPONENT libopenscenegraph-dev
   )
ENDFOREACH( INCLUDEFILE)
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to