Skip to content

Commit

Permalink
kbuild: rust-analyzer: mark rust_is_available.sh invocation as recu…
Browse files Browse the repository at this point in the history
…rsive

When calling the `rust_is_available.sh` script, we need to make the
jobserver available to it, as commit ecab411 ("kbuild: mark `rustc`
(and others) invocations as recursive") explains and did for the others.

Otherwise, we get a warning from `rustc` when calling `make rust-analyzer`
with parallel jobs, e.g. `-j8`. Using several jobs for that target does
not really matter, but developers may call `make` with jobs enabled in
all cases.

Thus fix it.

Fixes: 6dc9d9c ("kbuild: rust-analyzer: better error handling")
Reviewed-by: Alice Ryhl <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
[ Reworded to add a couple more details mentioned in the list. - Miguel ]
Signed-off-by: Miguel Ojeda <[email protected]>
  • Loading branch information
ojeda committed Aug 9, 2024
1 parent 02dfd63 commit d734422
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -1963,7 +1963,7 @@ tags TAGS cscope gtags: FORCE
# Protocol).
PHONY += rust-analyzer
rust-analyzer:
$(Q)$(CONFIG_SHELL) $(srctree)/scripts/rust_is_available.sh
+$(Q)$(CONFIG_SHELL) $(srctree)/scripts/rust_is_available.sh
$(Q)$(MAKE) $(build)=rust $@

# Script to generate missing namespace dependencies
Expand Down

0 comments on commit d734422

Please sign in to comment.