Hendrik, On Mon, Nov 2, 2009 at 3:19 PM, Hendrik Sattler <p...@hendrik-sattler.de> wrote: > Zitat von Mathieu Malaterre <mathieu.malate...@gmail.com>: > >> I updated the FindJava.cmake. It now implements the VERSION* stuff, >> and I fixed the naming convention as per the readme.txt. I tested on >> linux/gcj, linux/sun-java and linux/openjdk-6 > > Why are you using NO_DEFAULT_PATH and but listing /usr/bin and > /usr/local/bin? This is wrong and breaks "/usr/local/bin before /usr/bin". > Just drop those from the list and also your NO_DEFAULT_PATH. If you want > your list first, use HINTS instead of PATHS.
Fixed. Thanks ! $ cvs ci -m"BUG: To be able to use ENV and at the same time allow /usr/loca/bin to override /usr/bin, apply patch suggested by Hendrik Sattler (thanks)" Committer: Mathieu Malaterre <mathieu.malate...@kitware.com> /cvsroot/CMake/CMake/Modules/FindJava.cmake,v <-- FindJava.cmake new revision: 1.22; previous revision: 1.21 > Additionally, you may want to mark the found values as advanced (like it was > before). Fixed also. Thanks again, -- Mathieu _______________________________________________ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://www.cmake.org/mailman/listinfo/cmake
