Commit 1b2714e
tests: use $(rootpaths) to get executable files paths for Bazel 8 compatibility (#2395)
In Bazel 8, the singular `$(rootpath)` expansions require that the
target expands to a
single file. The py rules have an unfortunate legacy behavior where
their default outputs
are the executable and the main py file, thus causing an error.
To fix, use the plural `$(rootpaths)`, then post-process the
space-separated string to get
just the executable portion of it.
Along the way...
* Add tests/integration/py_cc_toolchain_registered/bazel-* symlink to
Bazel ignore.
This avoids an infinite symlink expansions error and performance issues
when those
symlinks exist.
Work towards #2378
---------
Co-authored-by: Ivo List <[email protected]>1 parent 91e5751 commit 1b2714e
File tree
4 files changed
+12
-7
lines changed- examples/bzlmod
- tests
4 files changed
+12
-7
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
| 29 | + | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
163 | 163 | | |
164 | 164 | | |
165 | 165 | | |
166 | | - | |
| 166 | + | |
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
| |||
173 | 173 | | |
174 | 174 | | |
175 | 175 | | |
176 | | - | |
| 176 | + | |
177 | 177 | | |
178 | 178 | | |
179 | 179 | | |
| |||
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
186 | | - | |
| 186 | + | |
187 | 187 | | |
188 | 188 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
20 | 24 | | |
21 | 25 | | |
22 | 26 | | |
| |||
0 commit comments