tree d97dc715bee7d2d68c17ccd89c1482668a84a713
parent 5cbb5b16c08f1832c569608126b27046e32e18ad
author Richard Levasseur <richardlev@gmail.com> 1757179838 -0700
committer GitHub <noreply@github.com> 1757179838 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJovG/PCRC1aQ7uu5UhlAAAjQoQAAXp0Y6jiG08UHmgoEYZ2i0D
 Ek9kv/7R4j5WiYgcFD6T1MjsyDAT8okXngoMkhmUWOz6bVT1zHrvX2YJCZ6U5xbR
 ioRZjB5ZhlGgOSRp+YB/kOcUBx33g10KBxnwuhbtlnnzqt0EjhKq0+sS5DxiLxtE
 AbiHJUA6NA3TbnUsl+A1jXpFOnklx7RIuzIbhPlbXYGUwQ7iX96xvpl0B2wooCjh
 FLswnjr/qDGSUwroRwbBPNmxarpjxaz/voPycm7q2vfPE11y11s/foScgEwifEqn
 VrU1awkS8ALAV8XMjG2Qo68zPe7CuEGCbLA9bKYiz+c7iBG0qfKy616HZcWMat76
 Z8niiX8dW/TBTNKNmdkSynpydL/Rp3SODezvxNdem6A0XYBp10g9Nn3nqMo0inAg
 zwECK/2IqgWJ72AUiGEoTbwAuyu0Q1hPQh+B4U01h5gslROfNENRWruP/7v7daYH
 sfvlMJRsY6D11/vadgyVZKNXywn6qeC22e2VJM7gz3NsZ0okCmceURThxRDT4+dW
 yJAY97/Wd5eSLdX5tkQ5PD/Cj4Bazd1SSofrk8/t4SckmhadP3e6BHJ1P8uMcxxj
 JaLHgkScNpoQIqI4BJpFu0MCfb/8lLNPdevzenr1rchapmJJGFQV3M38e+XON53e
 nHqkDcv+rQv8HrpopaxA
 =Olct
 -----END PGP SIGNATURE-----
 

fix(system_python): write import paths to generated file instead of using PYTHONPATH (#3242)

This changes the system_python bootstrap to use a 2-stage process like
the script
bootstrap does. Among other things, this means the import paths are
written to
a generated file (`bazel_site_init.py`, same as boostrap=script) and
sys.path
setup is performed by the Python code in stage 2.

Since the PYTHONPATH environment variable isn't used, this fixes the
problem on
Windows where the value is too long.

This also better unifies the system_python and script based bootstraps
because the
same stage 2 code and bazel_site_init code is used.

Along the way, several other improvements:

* Fixes path ordering for system_python. The order now matches venv
ordering
  (stdlib, binary paths, runtime site packages).
* Makes the venv-based solution work when the site module is disabled
(`-S`).
* Makes `interpreter_args` attribute and
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` env var work with
system_python.
* Makes `main_module` work with system_python.
* Progress towards a supportable non-shell based bootstrap (a user
requested
this because their environment doesn't install any shells as a security
  precaution).

Fixes https://github.com/bazel-contrib/rules_python/issues/2652