Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add //python:none as public target to disable exec_interpreter #2226

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

rickeylev
Copy link
Contributor

When writing the toolchain docs, I realized there wasn't a public target to use for
disabling the exec_interpreter.

Fixed by adding an alias to the internal target.

Along the way:

  • Add the exec tools and cc toolchains to the doc gen
  • A few improvements to the cc/exec tools docs
  • Add public bzl file for py_exec_tools_toolchain and PyExecToolsInfo
  • Fix a bug in sphinx_bzl where local names were hiding global names even if the
    requested type didn't match (e.g. a macro foo referring to rule foo)
  • Fix xrefs in the python/cc/index.md; it wasn't setting the default domain to bzl
  • Fix object type definition for attributes: the object type name was "attribute",
    but everything else was using "attr"; switched to "attr"

@rickeylev rickeylev marked this pull request as ready for review September 16, 2024 21:24
@rickeylev rickeylev added this pull request to the merge queue Sep 17, 2024
Merged via the queue into bazelbuild:main with commit 5a856e5 Sep 17, 2024
4 checks passed
@rickeylev rickeylev deleted the docs.toolchains branch October 29, 2024 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants