Milan Vukov

Results 88 comments of Milan Vukov

OK, will take a look at that as well. Thanks for helping out. Anyways, it would be nice to have this working out of the box eventually :).

It turns out, the error is on my end, per documentation of [subprocess.run](https://docs.python.org/3/library/subprocess.html#subprocess.run). If env is specified it overrides the caller process env. So, something like to following fixes the...

Well, at first the behavior of rules_py in this case was unexpected (works with rules_python :) ). Then I learned that my code was incomplete. Once I fixed it, then...

I think the following entry is missing in the path: ``` "/home/mvukov/.cache/bazel/_bazel_mvukov/56518e08ebcaaca6f37ddc3196658013/execroot/import_path_bug/bazel-out/k8-fastbuild/bin/libs/bar/bar.runfiles/import_path_bug", ```

The issue is in https://github.com/aspect-build/rules_py/blob/main/py/private/venv/venv.bzl#L58. If I extend the `imports_depset` as: ```Python imports_depset = depset( direct = [ctx.workspace_name], transitive = [imports_depset], ) ``` , then the provided example works as...

So, the current situation is that (as explained above) we have these entries in the Python path ``` ... .../bar.runfiles/import_path_bug/libs/foo .../bar.runfiles/import_path_bug/libs/bar ``` but the repo root is missing `.../bar.runfiles/import_path_bug` --...

@flolu Sorry, just saw this msg. You can use oqton remote, or, just download that commit, export as a patch, and patch the http_archive (if you want to use the...

I'd like to understand what's sub-optimal with my solution in https://github.com/aspect-build/rules_ts/pull/280. In the problematic cases that I experienced, the situation is pretty much simple: A depends on B, B depends...

Thanks, @gregmagolan ! Highly appreciated. OK, splitting devs/dev_deps makes sense now and should be more in-line how non-Bazel js tooling works. (IIUC, that should also reduce the amount of .d.ts...

There is a simple fact, but not immediately obvious. When you compile orocos and friends, you must copy the install space to the board to exactly the same path (at...