Hello,<br><br>Regarding the FindJava.cmake, I have a couple of improvements to suggest:<br><br>1) I have noticed that there is no JAVA_FOUND variable in the current version (and your version). <br><br>2) a JAVA_PATH (or another name) could also return the path where the Java executables are located.<br>
<br>Since I do not know the good practices to write such modules, do you think it would make sense to add those variables?<br><br>Cheers,<br><br>Romain Chanu<br><br><div class="gmail_quote">2009/10/30 Mathieu Malaterre <span dir="ltr"><<a href="mailto:mathieu.malaterre@gmail.com">mathieu.malaterre@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I updated the FindJava.cmake. It now implements the VERSION* stuff,<br>
and I fixed the naming convention as per the readme.txt. I tested on<br>
linux/gcj, linux/sun-java and linux/openjdk-6<br>
<br>
Cheers<br>
--<br>
Mathieu<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" target="_blank">http://www.cmake.org/Wiki/CMake_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.cmake.org/mailman/listinfo/cmake" target="_blank">http://www.cmake.org/mailman/listinfo/cmake</a><br>
</blockquote></div><br>