Jonathon Belotti
Jonathon Belotti
@jxramos RE the "forces the debug session to quit" I've copied below a workaround: > ... Unfortunately, this does not work so simply for `bazel test` and `py_test` targets. If...
@pstradomski I tested using `py_test` and `pdb` and it did not work for me. If it actually does work, then I'd certainly be interested in finding out what was wrong...
> but then I don't understand how changing to py_binary would help. I don't either, but it does help. Didn't bother venturing into the source to understand what was up.
Some work has begun here on `pytest` support: https://github.com/bazelbuild/rules_python/pull/464
Thanks for the report. This same problem is currently documented here https://github.com/bazelbuild/bazel/issues/7091 but I cannot find a counterpart already in this project. The Python stub template has an example of...
I think this is a duplicate of https://github.com/bazelbuild/bazel/issues/8446? Agree that the `README` could make clearer the stub script `python` dependency. @mattem has Aspect successfully used https://github.com/bazelbuild/bazel/commit/763dd0ce6e1644bf895231432f616427a11d385a anywhere? It appears to...
`python_register_toolchains` sets up the `py_runtime` on behalf of the user, so it would have to be responsible for detecting that the Bazel version is high enough and then adding the...
Just commenting in an in-the-wild example of `stub_shebang` usage that I found linked in the Bazel Slack: https://github.com/SeleniumHQ/selenium/commit/210bcf5d04c102f96cbb847790625eed1d829e39#diff-7fc57714ef13c3325ce2a1130202edced92fcccc0c6db34a72f7b57f60d552a3R20 That codebase is on Bazel `5.1.1` and so can take advantage of...
@mattem @f0rmiga @hrfuller I've thrown up a draft PR of using `stub_shebang` and doing version checking: https://github.com/bazelbuild/rules_python/pull/699. I've added some commentary on it too. Works with my very basic `bazel...
@lummax does this basic approach shown in https://thundergolfer.com/bazel/python/2021/06/25/a-basic-python-bazel-toolchain/ not work for your use-case? In a `.bzl` file you just have conditionals and grab the platform type from `repository_ctx.os.name.lower()`.