Title: [41279] trunk/hudson/plugins/vncviewer/src/main/java/org/jenkinsci/plugins/vncviewer/VncViewerBuildWrapper.java: removed unused code
Revision
41279
Author
tbaum
Date
2015-03-26 08:20:12 -0400 (Thu, 26 Mar 2015)

Log Message

removed unused code

Modified Paths


Diff

Modified: trunk/hudson/plugins/vncviewer/src/main/java/org/jenkinsci/plugins/vncviewer/VncViewerBuildWrapper.java (41278 => 41279)


--- trunk/hudson/plugins/vncviewer/src/main/java/org/jenkinsci/plugins/vncviewer/VncViewerBuildWrapper.java	2015-03-26 12:13:04 UTC (rev 41278)
+++ trunk/hudson/plugins/vncviewer/src/main/java/org/jenkinsci/plugins/vncviewer/VncViewerBuildWrapper.java	2015-03-26 12:20:12 UTC (rev 41279)
@@ -45,7 +45,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.InetAddress;
-import java.util.Map;
 
 import net.sf.json.JSONObject;
 
@@ -64,7 +63,6 @@
 	public VncViewerBuildWrapper(String vncServ) 
 	{
 		this.vncServ = vncServ;
-		//		this.vncServ = "10.137.12.222:5900"; //TODO
 	}
 
 	public String getVncServ() {
@@ -74,16 +72,16 @@
 	public void setVncServ(String vncServ) {
 		this.vncServ = vncServ;
 	}
-	
+
 	@Override
 	public Environment setUp(@SuppressWarnings("rawtypes")AbstractBuild build, Launcher launcher,
 			final BuildListener listener) throws IOException, InterruptedException
 	{
 		DescriptorImpl DESCRIPTOR = Hudson.getInstance().getDescriptorByType(DescriptorImpl.class);
 		String vncServReplaced = Util.replaceMacro(vncServ,build.getEnvironment(listener));
-		int localPort = 8888;
+		int startPortNmb = 8888;
 		Proc noVncProc = null;
-		String lp = String.valueOf(localPort);
+		String lp = String.valueOf(startPortNmb);
 		final ByteArrayOutputStream loggingStream = new ByteArrayOutputStream();
 		if (vncServReplaced.isEmpty())
 			vncServReplaced = DESCRIPTOR.getDefaultVncServ();
@@ -96,7 +94,7 @@
 		{
 			vncServReplaced = vncServReplaced.replace(":", ":59");
 		}
-		
+
 		try {
 			untar(VncViewerBuildWrapper.class.getResourceAsStream("/novnc.tar"),System.getProperty("java.io.tmpdir"));
 			untar(VncViewerBuildWrapper.class.getResourceAsStream("/websockify.tar"),System.getProperty("java.io.tmpdir"));
@@ -110,7 +108,7 @@
 			LocalLauncher localLauncher = new LocalLauncher(listener);
 			for (int i = 0; i < 1000 ; i++ )
 			{
-				lp = String.valueOf(localPort + i);
+				lp = String.valueOf(startPortNmb + i);
 				noVncProc = localLauncher.launch().stderr(loggingStream).stdout(loggingStream).cmds(webSockifyPath, "--web", webPath,lp,vncServReplaced).start();
 				Thread.sleep(5000);
 				if (noVncProc.isAlive())
@@ -122,8 +120,7 @@
 					try {noVncProc.kill();}catch (Exception e){} 
 				}
 			}
-			
-//			startNoVnc(build,listener,launcher);
+
 		} catch (Exception e) {
 			e.printStackTrace();
 		}
@@ -138,19 +135,11 @@
 		final Proc noVncProcFinal = noVncProc;
 		return new Environment() {
 			@Override
-			public void buildEnvVars(Map<String, String> env) {
-				//				env.put("PATH",env.get("PATH"));
-				//				env.put("DISPLAY", vncServReplaced);
-			}
-			@Override
 			public boolean tearDown(AbstractBuild build, BuildListener listener)
 					throws IOException, InterruptedException {
-
-
 				try {
 
-					ExpandableDetailsNote dn = new ExpandableDetailsNote("VNC viewer logs",loggingStream.toString());
-					listener.annotate(dn);
+					listener.annotate(new ExpandableDetailsNote("VNC viewer logs",loggingStream.toString()));
 					listener.getLogger().print('\n');
 					noVncProcFinal.kill();
 					loggingStream.close();
@@ -162,14 +151,13 @@
 		};
 	}
 
-	@Extension(ordinal = -1)
+	@Extension(ordinal = -2)
 	public static final class DescriptorImpl extends BuildWrapperDescriptor {
 		public DescriptorImpl() {
 			super(VncViewerBuildWrapper.class);
 			load();
 		}
 
-
 		@Override
 		public boolean configure(StaplerRequest req, JSONObject json) throws FormException {
 			req.bindJSON(this,json);
@@ -206,7 +194,6 @@
 	}
 
 	public static void untar(InputStream is, String dest) throws IOException {
-		//		  logger.info("Untar {}.",tarFile.getAbsolutePath());
 		TarInputStream tarIn = new TarInputStream(is);
 		try {
 			TarEntry tarEntry = tarIn.getNextEntry();
@@ -217,9 +204,11 @@
 					if (tarEntry.isDirectory()) 
 					{
 						destPath.mkdirs();
+						destPath.deleteOnExit();
 					} else 
 					{
 						destPath.createNewFile();
+						destPath.deleteOnExit();
 						byte [] btoRead = new byte[1024];
 						BufferedOutputStream bout = 
 								new BufferedOutputStream(new FileOutputStream(destPath));
@@ -229,7 +218,6 @@
 						{
 							bout.write(btoRead,0,len);
 						}
-
 						bout.close();
 						btoRead = null;
 					}

--
You received this message because you are subscribed to the Google Groups "Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to