]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit - scripts/Kbuild.include
kbuild: use -- separater intead of $(filter-out ...) for cc-cross-prefix
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Tue, 9 Jul 2019 06:13:00 +0000 (15:13 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 11 Jul 2019 14:34:52 +0000 (23:34 +0900)
commitd4a74bbfee03acf7bbddc77b9c9236462c744fc7
tree9303e6c70eb8616581ce0f8ca9d88268b71dd9b3
parent3a475b2166fd6aa5ac76e8c605dffdf7a2a457ee
kbuild: use -- separater intead of $(filter-out ...) for cc-cross-prefix

arch/mips/Makefile passes prefixes that start with '-' to cc-cross-prefix
when $(tool-archpref) evaluates to the empty string.

They are filtered-out before the $(shell ...) invocation. Otherwise,
'command -v' would be confused.

  $ command -v -linux-gcc
  bash: command: -l: invalid option
  command: usage: command [-pVv] command [arg ...]

Since commit 913ab9780fc0 ("kbuild: use more portable 'command -v' for
cc-cross-prefix"), cc-cross-prefix throws away the stderr output, so
the console is not polluted in any way.

This is not a big deal in practice, but I see a slightly better taste
in adding '--' to teach it that '-linux-gcc' is an argument instead of
a command option.

This will cause extra forking of subshell, but it will not be noticeable
performance regression.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
scripts/Kbuild.include