tree 548f21ed996c7c9f1839f11b8d7ab4f7a8cb86e2
parent e70d9e9e12d82890d41e4d0edcab4d722aad9ff0
author wyv <wyv@google.com> 1630341528 -0700
committer Copybara-Service <copybara-worker@google.com> 1630341646 -0700

bzlmod: create a base class for repository_ctx

(https://github.com/bazelbuild/bazel/issues/13316)

This base class contains methods to be shared between repository_ctx and module_ctx. In other words, the following methods are moved from repository_ctx to the new StarlarkBaseExternalContext:

* `path`: needed to traverse the file system
* `os`: needed for OS-specific information and env variables
* `read`: needed to read files
* `execute`: needed to perform arbitrary operations
* `which`: needed to locate tools on PATH

The other methods are left out because they're mostly useful for creating files, which module extensions are discouraged from (since the files created in such a way aren't accessible later, unlike those created by repo rules). We'd ideally also need an "http_get" method later.

This CL also changes the handling of marker data in repository_ctx (now in the base class), so that instead of directly writing entries into a marker data map, we collect those entries into a separate map. After the context object is used, that separate map can then be read and processed (so repository_ctx can put it into marker data and module_ctx will likely want to write it into the lockfile).

PiperOrigin-RevId: 393790084