clarify comment, remove commented out debug code

Author: rickeylev

python/private/py_executable_bazel.bzl

@@ -363,8 +363,10 @@ def _create_venv(ctx, output_prefix, imports, runtime_details):
     if runtime.interpreter:
         py_exe_basename = paths.basename(runtime.interpreter.short_path)
 
-        # A real symlink is required to support RBE. RBE may materialize what
-        # ctx.actions.symlink() points to.
+        # Even though ctx.actions.symlink() is used, using
+        # declare_symlink() is required to ensure that the resulting file
+        # in runfiles is always a symlink. An RBE implementation, for example,
+        # may choose to write what symlink() points to instead.
         interpreter = ctx.actions.declare_symlink("{}/bin/{}".format(venv, py_exe_basename))
         interpreter_actual_path = runtime.interpreter.short_path
         parent = "/".join([".."] * (interpreter_actual_path.count("/") + 1))

python/private/stage1_bootstrap_template.sh

@@ -128,12 +128,6 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
   # The bin/ directory may not exist if it is empty.
   mkdir -p "$(dirname $python_exe)"
   ln -s "$symlink_to" "$python_exe"
-else
-  :
-  #set -x
-  #stat "$python_exe"
-  #ls -l $(dirname $python_exe)
-  #set +x
 fi
 
 # At this point, we should have a valid reference to the interpreter.