No this is ok as-is. On 2011-04-30, Amit Kulkarni <amitk...@gmail.com> wrote: > Index: patch-Modules_CMakeDetermineJavaCompiler_cmake >=================================================================== > RCS file: > /cvs/ports/devel/cmake/patches/patch-Modules_CMakeDetermineJavaCompiler_cmake,v > retrieving revision 1.1 > diff -u patch-Modules_CMakeDetermineJavaCompiler_cmake > --- patch-Modules_CMakeDetermineJavaCompiler_cmake 28 May 2010 20:21:48 > -0000 1.1 > +++ patch-Modules_CMakeDetermineJavaCompiler_cmake 30 Apr 2011 01:47:34 > -0000 > @@ -41,7 +41,7 @@ > + SET(Java_BIN_PATH /usr/local/jdk-1.5.0/bin) > + ELSEIF(MODJAVA_VER MATCHES "^1\\.6.*") > + SET(Java_BIN_PATH /usr/local/jdk-1.6.0/bin) > -+ ELSE(MODJAVA_VER MATCHES "^1\\.5.*")
This text in ELSE(...) matches the text in the IF line. If there's a version of java newer than 1.7 the ELSE will need to be changed to ELSEIF and another check added but it's fine for now.