← Back to team overview

zorba-coders team mailing list archive

[Merge] lp:~zorba-coders/zorba/bug1053736 into lp:zorba

 

Rodolfo Ochoa has proposed merging lp:~zorba-coders/zorba/bug1053736 into lp:zorba.

Commit message:
Changing conditional, seems that this is an error on windows

Requested reviews:
  Rodolfo Ochoa (rodolfo-ochoa)
Related bugs:
  Bug #1053736 in Zorba: "JavaTest should be out if JDK is not installed"
  https://bugs.launchpad.net/zorba/+bug/1053736

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1053736/+merge/126088

Changing conditional, seems that this is an error on windows
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1053736/+merge/126088
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'test/rbkt/modules/java/CMakeLists.txt'
--- test/rbkt/modules/java/CMakeLists.txt	2012-09-17 00:36:37 +0000
+++ test/rbkt/modules/java/CMakeLists.txt	2012-09-24 20:22:20 +0000
@@ -19,7 +19,7 @@
 COMPARE_VERSION_STRINGS (${CMAKE_VERSION} "2.8.6" RESULT)
 IF (${RESULT} GREATER -1)
   FIND_PACKAGE (Java COMPONENTS Development)
-  IF (Java_Development_FOUND)
+  IF (Java_JAVAC_EXECUTABLE)
     INCLUDE (UseJava)
 
     # Declare two "different" jars, for testing


Follow ups