Bazel client: delete and don't use the server.pid

Upon startup the Bazel client checks if there's
already a running server process and if so then
connects to it.

We achieve this by checking if there's a symlink
in the server directory called served.pid,
pointing to /proc/<server_pid>. If so, we read the
symlink's target and extract the PID; otherwise we
check if there's a file in the server's directory
(server.pid.txt) that contains the PID and read it
from there.

Since the PID file is always there, we don't need
the symlink, plus on Windows we don't support
symlinks anyway, which is the real motivation for
this change.

Just ignoring the PID symlink is not enough, we
need to actively delete it so that switching
between Bazel versions (one that writes a PID
symlink and one that doesn't) won't result in
having a symlink and PID file with different
PIDs and clients trying to kill the wrong server
process / not killing one that they should.

See https://github.com/bazelbuild/bazel/issues/2107

--
MOS_MIGRATED_REVID=140117287

1 files changed, 0 insertions, 6 deletions

diff --git a/src/main/cpp/blaze_util_posix.cc b/src/main/cpp/blaze_util_posix.cc
index 55cb59b563..b5d8caf47e 100644
--- a/src/main/cpp/blaze_util_posix.cc
+++ b/src/main/cpp/blaze_util_posix.cc
@@ -160,7 +160,6 @@ void ExecuteDaemon(const string& exe,
   Daemonize(daemon_output);
   string pid_string = GetProcessIdAsString();
   string pid_file = blaze_util::JoinPath(server_dir, kServerPidFile);
-  string pid_symlink_file = blaze_util::JoinPath(server_dir, kServerPidSymlink);
 
   if (!WriteFile(pid_string, pid_file)) {
     // The exit code does not matter because we are already in the daemonized
@@ -168,11 +167,6 @@ void ExecuteDaemon(const string& exe,
     pdie(0, "Cannot write PID file");
   }
 
-  UnlinkPath(pid_symlink_file.c_str());
-  if (symlink(pid_string.c_str(), pid_symlink_file.c_str()) < 0) {
-    pdie(0, "Cannot write PID symlink");
-  }
-
   WriteSystemSpecificProcessIdentifier(server_dir);
 
   ExecuteProgram(exe, args_vector);