[cmake-developers] Java support

[CHEVRIER, Marc]

Hi,

New version of patch attached.
I added a check on java tool to execute test only if java and shared library built by test have same architecture (32 or 64bit).

Marc




On 05/08/15 17:49, "Brad King" <brad.king at kitware.com> wrote:

>On 08/05/2015 03:15 AM, CHEVRIER, Marc wrote:
>> For point 2., The best approach is to install 32 and 64bit
>> versions of Java SDK (Windows versions, not some cygwin or
>> MSYS versions) and ensure JAVA_HOME is NOT set in the
>> environment nor referenced in the PATH. In this case, CMake
>> FindJava.cmake module will look at the registry to retrieve
>> the correct java version.
>
>CMake's test suite needs to adapt to the environment in which
>it is run and not run this test unless the architecture of
>java and the toolchain match.  Certainly we can set up an
>environment in which it works but we should not fail just
>because that hasn't been done.
>
>On 08/05/2015 06:24 AM, CHEVRIER, Marc wrote:
>> Again, a new version for patch 4.
>
>Thanks.  I'd prefer not to enable CXX as a language in the
>project() command such that it affects the other Java tests.
>Please create a separate source tree for the new test.  Sorry
>I didn't notice this earlier.
>
>Meanwhile the other three patches now tested cleanly.
>
>Thanks,
>-Brad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Add-support-for-javah-tool.patch
Type: application/octet-stream
Size: 10356 bytes
Desc: 0004-Add-support-for-javah-tool.patch
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20150807/d99e2cbf/attachment-0001.obj>