author | alundblad |
Tue, 25 Aug 2015 15:14:41 +0200 | |
changeset 32335 | 7df616378cf3 |
parent 31751 | ec251536a004 |
child 32542 | f4e4f4c4f9f4 |
permissions | -rw-r--r-- |
25605
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
1 |
/* |
32335 | 2 |
* Copyright (c) 2014, 2015, Oracle and/or its affiliates. All rights reserved. |
25605
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
4 |
* |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
7 |
* published by the Free Software Foundation. Oracle designates this |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
8 |
* particular file as subject to the "Classpath" exception as provided |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
10 |
* |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
15 |
* accompanied this code). |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
16 |
* |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
17 |
* You should have received a copy of the GNU General Public License version |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
20 |
* |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
22 |
* or visit www.oracle.com if you need additional information or have any |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
23 |
* questions. |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
24 |
*/ |
c2d72342fe19
8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents:
25299
diff
changeset
|
25 |
|
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
26 |
package com.sun.tools.sjavac.client; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
27 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
28 |
import java.io.File; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
29 |
import java.io.IOException; |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
30 |
import java.io.ObjectInputStream; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
31 |
import java.io.ObjectOutputStream; |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
32 |
import java.io.PrintStream; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
33 |
import java.net.InetAddress; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
34 |
import java.net.InetSocketAddress; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
35 |
import java.net.Socket; |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
36 |
import java.util.ArrayList; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
37 |
import java.util.Arrays; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
38 |
import java.util.List; |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
39 |
import java.util.Scanner; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
40 |
|
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
41 |
import com.sun.tools.sjavac.Log; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
42 |
import com.sun.tools.sjavac.Util; |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
43 |
import com.sun.tools.sjavac.options.OptionHelper; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
44 |
import com.sun.tools.sjavac.options.Options; |
32335 | 45 |
import com.sun.tools.sjavac.server.CompilationSubResult; |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
46 |
import com.sun.tools.sjavac.server.CompilationResult; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
47 |
import com.sun.tools.sjavac.server.PortFile; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
48 |
import com.sun.tools.sjavac.server.Sjavac; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
49 |
import com.sun.tools.sjavac.server.SjavacServer; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
50 |
|
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
51 |
/** |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
52 |
* Sjavac implementation that delegates requests to a SjavacServer. |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
53 |
* |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
54 |
* <p><b>This is NOT part of any supported API. |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
55 |
* If you write code that depends on this, you do so at your own risk. |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
56 |
* This code and its internal interfaces are subject to change or |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
57 |
* deletion without notice.</b> |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
58 |
*/ |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
59 |
public class SjavacClient implements Sjavac { |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
60 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
61 |
// The id can perhaps be used in the future by the javac server to reuse the |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
62 |
// JavaCompiler instance for several compiles using the same id. |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
63 |
private final String id; |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
64 |
private final PortFile portFile; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
65 |
private final String logfile; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
66 |
private final String stdouterrfile; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
67 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
68 |
// Default keepalive for server is 120 seconds. |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
69 |
// I.e. it will accept 120 seconds of inactivity before quitting. |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
70 |
private final int keepalive; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
71 |
private final int poolsize; |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
72 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
73 |
// The sjavac option specifies how the server part of sjavac is spawned. |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
74 |
// If you have the experimental sjavac in your path, you are done. If not, you have |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
75 |
// to point to a com.sun.tools.sjavac.Main that supports --startserver |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
76 |
// for example by setting: sjavac=java%20-jar%20...javac.jar%com.sun.tools.sjavac.Main |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
77 |
private final String sjavacForkCmd; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
78 |
|
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
79 |
// Wait 2 seconds for response, before giving up on javac server. |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
80 |
static int CONNECTION_TIMEOUT = 2000; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
81 |
static int MAX_CONNECT_ATTEMPTS = 3; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
82 |
static int WAIT_BETWEEN_CONNECT_ATTEMPTS = 2000; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
83 |
|
26088 | 84 |
// Store the server conf settings here. |
85 |
private final String settings; |
|
86 |
||
32335 | 87 |
public SjavacClient(Options options) throws PortFileInaccessibleException { |
26088 | 88 |
String tmpServerConf = options.getServerConf(); |
89 |
String serverConf = (tmpServerConf!=null)? tmpServerConf : ""; |
|
90 |
String tmpId = Util.extractStringOption("id", serverConf); |
|
91 |
id = (tmpId!=null) ? tmpId : "id"+(((new java.util.Random()).nextLong())&Long.MAX_VALUE); |
|
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
92 |
String defaultPortfile = options.getStateDir() |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
93 |
.resolve("javac_server") |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
94 |
.toAbsolutePath() |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
95 |
.toString(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
96 |
String portfileName = Util.extractStringOption("portfile", serverConf, defaultPortfile); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
97 |
try { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
98 |
portFile = SjavacServer.getPortFile(portfileName); |
32335 | 99 |
} catch (PortFileInaccessibleException e) { |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
100 |
Log.error("Port file inaccessable: " + e); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
101 |
throw e; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
102 |
} |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
103 |
logfile = Util.extractStringOption("logfile", serverConf, portfileName + ".javaclog"); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
104 |
stdouterrfile = Util.extractStringOption("stdouterrfile", serverConf, portfileName + ".stdouterr"); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
105 |
sjavacForkCmd = Util.extractStringOption("sjavac", serverConf, "sjavac"); |
26088 | 106 |
int poolsize = Util.extractIntOption("poolsize", serverConf); |
107 |
keepalive = Util.extractIntOption("keepalive", serverConf, 120); |
|
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
108 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
109 |
this.poolsize = poolsize > 0 ? poolsize : Runtime.getRuntime().availableProcessors(); |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
110 |
settings = (serverConf.equals("")) ? "id="+id+",portfile="+portfileName : serverConf; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
111 |
} |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
112 |
|
26088 | 113 |
/** |
114 |
* Hand out the server settings. |
|
115 |
* @return The server settings, possibly a default value. |
|
116 |
*/ |
|
117 |
public String serverSettings() { |
|
118 |
return settings; |
|
119 |
} |
|
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
120 |
|
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
121 |
@Override |
32335 | 122 |
public CompilationResult compile(String[] args) { |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
123 |
CompilationResult result; |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
124 |
try (Socket socket = tryConnect()) { |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
125 |
// The ObjectInputStream constructor will block until the |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
126 |
// corresponding ObjectOutputStream has written and flushed the |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
127 |
// header, so it is important that the ObjectOutputStreams on server |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
128 |
// and client are opened before the ObjectInputStreams. |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
129 |
ObjectOutputStream oos = new ObjectOutputStream(socket.getOutputStream()); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
130 |
ObjectInputStream ois = new ObjectInputStream(socket.getInputStream()); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
131 |
oos.writeObject(id); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
132 |
oos.writeObject(SjavacServer.CMD_COMPILE); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
133 |
oos.writeObject(args); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
134 |
oos.flush(); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
135 |
result = (CompilationResult) ois.readObject(); |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
136 |
} catch (IOException | ClassNotFoundException ex) { |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
137 |
Log.error("[CLIENT] Exception caught: " + ex); |
32335 | 138 |
result = new CompilationResult(CompilationSubResult.ERROR_FATAL); |
26991
88d998b3bb4b
8056258: Analysis of public API does not take super classes into account
alundblad
parents:
26107
diff
changeset
|
139 |
result.stderr = Util.getStackTrace(ex); |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
140 |
} catch (InterruptedException ie) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
141 |
Thread.currentThread().interrupt(); // Restore interrupt |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
142 |
Log.error("[CLIENT] compile interrupted."); |
32335 | 143 |
result = new CompilationResult(CompilationSubResult.ERROR_FATAL); |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
144 |
result.stderr = Util.getStackTrace(ie); |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
145 |
} |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
146 |
return result; |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
147 |
} |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
148 |
|
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
149 |
/* |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
150 |
* Makes MAX_CONNECT_ATTEMPTS attepmts to connect to server. |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
151 |
*/ |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
152 |
private Socket tryConnect() throws IOException, InterruptedException { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
153 |
makeSureServerIsRunning(portFile); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
154 |
int attempt = 0; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
155 |
while (true) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
156 |
Log.info("Trying to connect. Attempt " + (++attempt) + " of " + MAX_CONNECT_ATTEMPTS); |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
157 |
try { |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
158 |
return makeConnectionAttempt(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
159 |
} catch (IOException ex) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
160 |
Log.error("Connection attempt failed: " + ex.getMessage()); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
161 |
if (attempt >= MAX_CONNECT_ATTEMPTS) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
162 |
Log.error("Giving up"); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
163 |
throw new IOException("Could not connect to server", ex); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
164 |
} |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
165 |
} |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
166 |
Thread.sleep(WAIT_BETWEEN_CONNECT_ATTEMPTS); |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
167 |
} |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
168 |
} |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
169 |
|
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
170 |
private Socket makeConnectionAttempt() throws IOException { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
171 |
Socket socket = new Socket(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
172 |
InetAddress localhost = InetAddress.getByName(null); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
173 |
InetSocketAddress address = new InetSocketAddress(localhost, portFile.getPort()); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
174 |
socket.connect(address, CONNECTION_TIMEOUT); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
175 |
Log.info("Connected"); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
176 |
return socket; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
177 |
} |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
178 |
|
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
179 |
/* |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
180 |
* Will return immediately if a server already seems to be running, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
181 |
* otherwise fork a new server and block until it seems to be running. |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
182 |
*/ |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
183 |
private void makeSureServerIsRunning(PortFile portFile) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
184 |
throws IOException, InterruptedException { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
185 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
186 |
portFile.lock(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
187 |
portFile.getValues(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
188 |
portFile.unlock(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
189 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
190 |
if (portFile.containsPortInfo()) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
191 |
// Server seems to already be running |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
192 |
return; |
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
193 |
} |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
194 |
|
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
195 |
// Fork a new server and wait for it to start |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
196 |
SjavacClient.fork(sjavacForkCmd, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
197 |
portFile, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
198 |
logfile, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
199 |
poolsize, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
200 |
keepalive, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
201 |
System.err, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
202 |
stdouterrfile); |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
203 |
} |
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
204 |
|
26098
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
205 |
@Override |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
206 |
public void shutdown() { |
32588700060b
8048457: Sjavac should not use portfiles, sockets, etc if background=false
alundblad
parents:
26088
diff
changeset
|
207 |
// Nothing to clean up |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
208 |
} |
27383
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
209 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
210 |
/* |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
211 |
* Fork a server process process and wait for server to come around |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
212 |
*/ |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
213 |
public static void fork(String sjavacCmd, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
214 |
PortFile portFile, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
215 |
String logfile, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
216 |
int poolsize, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
217 |
int keepalive, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
218 |
final PrintStream err, |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
219 |
String stdouterrfile) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
220 |
throws IOException, InterruptedException { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
221 |
List<String> cmd = new ArrayList<>(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
222 |
cmd.addAll(Arrays.asList(OptionHelper.unescapeCmdArg(sjavacCmd).split(" "))); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
223 |
cmd.add("--startserver:" |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
224 |
+ "portfile=" + portFile.getFilename() |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
225 |
+ ",logfile=" + logfile |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
226 |
+ ",stdouterrfile=" + stdouterrfile |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
227 |
+ ",poolsize=" + poolsize |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
228 |
+ ",keepalive="+ keepalive); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
229 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
230 |
Process p = null; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
231 |
Log.info("Starting server. Command: " + String.join(" ", cmd)); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
232 |
try { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
233 |
// If the cmd for some reason can't be executed (file not found, or |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
234 |
// is not executable) this will throw an IOException with a decent |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
235 |
// error message. |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
236 |
p = new ProcessBuilder(cmd) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
237 |
.redirectErrorStream(true) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
238 |
.redirectOutput(new File(stdouterrfile)) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
239 |
.start(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
240 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
241 |
// Throws an IOException if no valid values materialize |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
242 |
portFile.waitForValidValues(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
243 |
|
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
244 |
} catch (IOException ex) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
245 |
// Log and rethrow exception |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
246 |
Log.error("Faild to launch server."); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
247 |
Log.error(" Message: " + ex.getMessage()); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
248 |
String rc = p == null || p.isAlive() ? "n/a" : "" + p.exitValue(); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
249 |
Log.error(" Server process exit code: " + rc); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
250 |
Log.error("Server log:"); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
251 |
Log.error("------- Server log start -------"); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
252 |
try (Scanner s = new Scanner(new File(stdouterrfile))) { |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
253 |
while (s.hasNextLine()) |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
254 |
Log.error(s.nextLine()); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
255 |
} |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
256 |
Log.error("------- Server log end ---------"); |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
257 |
throw ex; |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
258 |
} |
716ed9a6d607
8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way
alundblad
parents:
27226
diff
changeset
|
259 |
} |
25299
b4a7dcd657f5
8048594: The sjavac client/server protocol should be hidden behind an interface
alundblad
parents:
diff
changeset
|
260 |
} |