langtools/src/jdk.compiler/share/classes/com/sun/tools/sjavac/client/SjavacClient.java
author alundblad
Tue, 25 Aug 2015 15:14:41 +0200
changeset 32335 7df616378cf3
parent 31751 ec251536a004
child 32542 f4e4f4c4f9f4
permissions -rw-r--r--
8048318: Refactor sjavac as a thin client Summary: Moved compilation logic from client to server. Reviewed-by: jlahoda
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
25605
c2d72342fe19 8050408: A few new Java src files for sjavac are missing copyright notices
jjg
parents: 25299
diff changeset
     1
/*
32335
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
     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
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
    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
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    84
    // Store the server conf settings here.
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    85
    private final String settings;
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    86
32335
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
    87
    public SjavacClient(Options options) throws PortFileInaccessibleException {
26088
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    88
        String tmpServerConf = options.getServerConf();
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    89
        String serverConf = (tmpServerConf!=null)? tmpServerConf : "";
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    90
        String tmpId = Util.extractStringOption("id", serverConf);
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
    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
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
    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
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   106
        int poolsize = Util.extractIntOption("poolsize", serverConf);
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   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
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   113
    /**
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   114
     * Hand out the server settings.
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   115
     * @return The server settings, possibly a default value.
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   116
     */
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   117
    public String serverSettings() {
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   118
        return settings;
f479ca655ba1 8054474: Add --state-dir=bar to sjavac
ohrstrom
parents: 25605
diff changeset
   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
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
   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
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
   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
7df616378cf3 8048318: Refactor sjavac as a thin client
alundblad
parents: 31751
diff changeset
   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
}