tree bf88a7c1c283592f4288e7d1520bcf9803eaa5fe
parent 053508ff010b6e21602fb7e8005e64a2ba4d536a
author brandjon <brandjon@google.com> 1550256973 -0800
committer Copybara-Service <copybara-worker@google.com> 1550257081 -0800

Refactor some py_runtime code

Moved its configured target factory, provider, and test to the ...lib/rules/python directory instead of ...lib/bazel/rules/python. Simplified the test. Also moved BazelPythonConfigurationTest into the bazel dir.

The `files` attribute of py_runtime is no longer mandatory. Since a non-empty value is not permitted when `interpreter_path` is used, it makes more sense to omit it rather than explicitly set it to empty.

The error messages and rule logic are simplified. The provider now has an invariant that either interpreter is null or else interpreterPath and files are null; this will be clarified in a follow-up CL that exposes PyRuntimeProvider to Starlark.

Work toward #7375.

PiperOrigin-RevId: 234174755