cleanup

Author: rickeylev

.bazelci/presubmit.yml

@@ -227,20 +227,18 @@ tasks:
     # for now until we can look into it.
     build_targets:
       - "--"
-      ##- "..."
-      ### As a regression test for #225, check that wheel targets still build when
-      ### their package path is qualified with the repo name.
-      ##- "@rules_python//examples/wheel/..."
-      - "//tests/py_zipapp:system_python_zipapp_test"
+      - "..."
+      # As a regression test for #225, check that wheel targets still build when
+      # their package path is qualified with the repo name.
+      - "@rules_python//examples/wheel/..."
     build_flags:
       - "--noenable_bzlmod"
       - "--enable_workspace"
       - "--keep_going"
       - "--build_tag_filters=-integration-test"
     test_targets:
       - "--"
-      ##- "..."
-      - "//tests/py_zipapp:system_python_zipapp_test"
+      - "..."
     test_flags:
       - "--noenable_bzlmod"
       - "--enable_workspace"
@@ -258,14 +256,8 @@ tasks:
     <<: *reusable_config
     name: "Default: Windows"
     platform: windows
-    build_targets:
-      - "--"
-      - "//tests/py_zipapp:system_python_zipapp_test"
     test_flags:
       - "--test_tag_filters=-integration-test,-fix-windows"
-    test_targets:
-      - "--"
-      - "//tests/py_zipapp:system_python_zipapp_test"
   rbe_min:
     <<: *minimum_supported_version
     <<: *reusable_config

python/private/zipapp/zip_main_template.py

@@ -275,7 +275,6 @@ def main():
         if not os.path.lexists(python_program):
             # The venv bin/python3 interpreter should always be under runfiles, but
             # double check. We don't want to accidentally create symlinks elsewhere
-            # or unlink outside our tree.
             if not python_program.startswith(module_space):
                 raise AssertionError(
                     "Program's venv binary not under runfiles: {python_program}"