Merged revisions 33168 via svnmerge from

https://www.dev.java.net/svn/hudson/branches/rc

........
  r33168 | kohsuke | 2010-07-22 10:42:12 -0700 (Thu, 22 Jul 2010) | 1 line
  
  fixed a silly compilation problem
........


git-svn-id: https://hudson.dev.java.net/svn/hudson/trunk/hudson/main@33169 71c3de6d-444a-0410-be80-ed276b4c234a

core/src/main/java/hudson/tools/JDKInstaller.java

@@ -162,10 +162,11 @@ public class JDKInstaller extends ToolInstaller {
         case LINUX:
         case SOLARIS:
             fs.chmod(jdkBundle,0755);
-            if (launcher.launch().cmds(jdkBundle, "-noregister")
-                  .stdin(new ByteArrayInputStream("yes".getBytes())).stdout(out)
-                  .pwd(new FilePath(launcher.getChannel(), expectedLocation)).join() != 0)
-                throw new AbortException(Messages.JDKInstaller_FailedToInstallJDK());
+            int exit = launcher.launch().cmds(jdkBundle, "-noregister")
+                    .stdin(new ByteArrayInputStream("yes".getBytes())).stdout(out)
+                    .pwd(new FilePath(launcher.getChannel(), expectedLocation)).join();
+            if (exit != 0)
+                throw new AbortException(Messages.JDKInstaller_FailedToInstallJDK(exit));
 
             // JDK creates its own sub-directory, so pull them up
             List<String> paths = fs.listSubDirectories(expectedLocation);