GitHub user snikulov opened a pull request:
https://github.com/apache/thrift/pull/365
cmake // build // fixed java library build enabler
Hello,
Here the fix for case when jdk is installed, but to Ant.
Since Ant is required to build java library, I move Ant search to
cmake/DefineOptions.cmake and add dependency for WITH_JAVA option.
Without those changes we'll pass JAVA_FOUND test and will fail later in
find_package(Ant REQUIRED)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snikulov/thrift cmake_java_lib_fix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/365.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #365
----
commit 87f3c86c97177b4737be766aca2f33990ee593e5
Author: Sergei Nikulov <[email protected]>
Date: 2015-01-30T10:29:33Z
Fixed Java lib build enable - Ant required to build java part of Thrift
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
