Commit 9b0214d
committed
Auto merge of rust-lang#66255 - ehuss:update-cc, r=alexcrichton
Update cc, git2, num_cpus.
This updates the `cc` crate, bringing in better parallel building support. Also updates `git2` which enables the parallel feature. (Note: I don't expect it will have a significant impact on build time, but seems good to update anyways.)
The main thorn is that `cc` gained knowledge about RISC-V architectures (https://github.com/alexcrichton/cc-rs/pull/428, https://github.com/alexcrichton/cc-rs/pull/429, https://github.com/alexcrichton/cc-rs/pull/430), but the builders on CI do not have the riscv C compiler installed. This means that bootstraps' cc detection was finding a C compiler that isn't installed, and fails.
The solution here is to override the cc detection to `false`. The C compiler isn't actually used on riscv platforms. AFAIK, the only location would be compiler_builtins, and it currently forces C support off (https://github.com/rust-lang/compiler-builtins/blob/a533ae9c5aa325db209659679535fe1f186eae81/build.rs#L49-L55).
Other possible solutions:
- Add the override in cc_detect for riscv (or any "no-C" platform like wasm32 and nvptx)
- Install and use the appropriate c compiler. I tried this the `g++-riscv64-linux-gnu` package, but it failed missing some header file.
Closes rust-lang#66232File tree
3 files changed
+21
-14
lines changed- src
- ci/docker/dist-various-1
- librustc
3 files changed
+21
-14
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
386 | 386 | | |
387 | 387 | | |
388 | 388 | | |
389 | | - | |
| 389 | + | |
390 | 390 | | |
391 | | - | |
| 391 | + | |
| 392 | + | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
392 | 396 | | |
393 | 397 | | |
394 | 398 | | |
| |||
1255 | 1259 | | |
1256 | 1260 | | |
1257 | 1261 | | |
1258 | | - | |
| 1262 | + | |
1259 | 1263 | | |
1260 | | - | |
| 1264 | + | |
1261 | 1265 | | |
1262 | 1266 | | |
1263 | 1267 | | |
| |||
1785 | 1789 | | |
1786 | 1790 | | |
1787 | 1791 | | |
1788 | | - | |
| 1792 | + | |
1789 | 1793 | | |
1790 | | - | |
| 1794 | + | |
1791 | 1795 | | |
1792 | 1796 | | |
1793 | 1797 | | |
| |||
2228 | 2232 | | |
2229 | 2233 | | |
2230 | 2234 | | |
2231 | | - | |
| 2235 | + | |
2232 | 2236 | | |
2233 | | - | |
| 2237 | + | |
2234 | 2238 | | |
2235 | 2239 | | |
2236 | 2240 | | |
| |||
3130 | 3134 | | |
3131 | 3135 | | |
3132 | 3136 | | |
3133 | | - | |
3134 | 3137 | | |
3135 | 3138 | | |
3136 | 3139 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
135 | 135 | | |
136 | 136 | | |
137 | 137 | | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
138 | 141 | | |
139 | 142 | | |
140 | 143 | | |
| |||
143 | 146 | | |
144 | 147 | | |
145 | 148 | | |
146 | | - | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
147 | 155 | | |
148 | 156 | | |
149 | 157 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | 13 | | |
18 | 14 | | |
19 | 15 | | |
| |||
0 commit comments