affects 544674 - vtk
severity 544674 important
thanks

Hello,

On ketvirtadienis 17 RugsÄ—jis 2009 10:05:57 Mathieu Malaterre wrote:
> Please apply upstream patch that should solve this grave issue.
> 
> Link:
> http://public.kitware.com/cgi-bin/viewcvs.cgi/Modules/FindJNI.cmake?root=CM
> ake&r1=1.40&r2=1.41

Mathieu, my previous arguments about your patch being arch-dependent still 
hold true. I somehow missed that you reported upstream bug and I would say it 
is rather unfortunate upstream accepted your patch. So I'm not going to rush 
but talk with upstream about proper solution.

Secondly, fixing #544674 bug will NOT fix #545335. Current FindJNI.cmake 
refers to $JAVA_HOME/ppc in the search path as it is supposed to (according to 
openjdk-6-jre-headless package). Don't you find it weird that vtk builds just 
*fine* with FindJNI.cmake on arches it supposedly does NOT support (according 
to this bug) but fails on powerpc which it supports properly? This bug 
obviously attracted too much false attention.

FWIW, from vtk debian/rules:

echo JAVA_AWT_LIBRARY:FILEPATH=/usr/lib/jvm/default-
java/jre/lib/$(DEB_HOST_ARCH_CPU)/libjawt.so >> Build/CMakeCache.txt

So that's where the powerpc FTBFS is coming from. It *overrides* proper value 
FindJNI sets on powerpc. It is also true that you cannot drop this workaround 
until #544674 is fixed. You will have to improve it.

-- 
Modestas Vainius <modes...@vainius.eu>

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to