On 04/27/2012 04:53 PM, Darrell Anderson wrote:
>> What is the status of the patch in
>> bug report 902?
>>
>> I tried the patch with the latest GIT and gcc 4.7. The build
>> still fails with the same error:
>>
>> collect2: error: ld returned 1 exit status
>> /usr/lib/gcc/i486-slackware-linux/4.7.0/../../../../i486-slackware-linux/bin/ld:
>> CMakeFiles/ksnapshot.dir/windowgrabber.cpp.o: undefined
>> reference to symbol 'XShapeQueryExtension'
> 
> The patch looks like this:
> 
> =================================================
> --- tdegraphics/ksnapshot/CMakeLists.txt
> +++ tdegraphics/ksnapshot/CMakeLists.txt	2012-03-09 09:18:44.692136349 -0600
> @@ -36,10 +36,20 @@
>  
>  ##### ksnapshot (executable) ####################
>  
> +## test for and explicitly include libXext
> +if( WITH_XEXT )
> +  pkg_search_module( XEXT xext )
> +  if( NOT XEXT_FOUND )
> +    tde_message_fatal( "You requested Xext support, but is was not found on 
> +your system (or devel package is not installed" )
> +  endif( )
> +  check_include_file( "X11/extensions/shape.h" HAVE_X11_EXTENSIONS_SHAPE_H )
> +endif( )
> +
>  tde_add_executable( ksnapshot AUTOMOC
>    SOURCES
>      ksnapshotiface.skel main.cpp ksnapshot.cpp regiongrabber.cpp
>      windowgrabber.cpp ksnapshotwidget.ui
> -  LINK tdeprint-shared
> +  LINK tdeprint-shared ${XEXT_LIBRARIES}
>    DESTINATION ${BIN_INSTALL_DIR}
>  )
> =================================================
> 
> The internal cmake variable XEXT_LIBRARIES is not being generated. How do we make that happen?
> 
> Darrell
> 

Sorry Darrell, was out tonight.

  I have used:

  # fix libXext inclusion in linker string (gcc >=4.6.2)
  msg "fixing libXext inclusion in linker string (gcc >=4.6.2)"
  sed -i "/LINK/s|$| Xext|" tdegraphics/ksnapshot/CMakeLists.txt

I don't know why I thought that had already been pushed...

-- 
David C. Rankin, J.D.,P.E.