diff options
author | Michael Forney <mforney@mforney.org> | 2018-08-16 02:13:50 +0200 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-08-22 16:21:40 +0200 |
commit | b5f8cb9e908f46feaa5d0ddbf3e2e1b8d85d1a65 (patch) | |
tree | 4df230f7f4489495f289af7bfcda7c07ff140272 /lib | |
parent | scripts: modpost: check memory allocation results (diff) | |
download | linux-b5f8cb9e908f46feaa5d0ddbf3e2e1b8d85d1a65.tar.xz linux-b5f8cb9e908f46feaa5d0ddbf3e2e1b8d85d1a65.zip |
kbuild: Add a space after `!` to prevent parsing as file pattern
Some shells use !(pattern|...|pattern) to match file names not
containing the specified patterns. This may result in output like
$ ./scripts/clang-version.sh gcc
./scripts/clang-version.sh[18]: COPYING: not found
printf: %d __clang_major__: conversion error
printf: %d __clang_minor__: conversion error
printf: %d __clang_patchlevel__: conversion error
00000
$
and set CONFIG_CLANG_VERSION to the invalid value '00000'.
POSIX says[0]
If the pipeline begins with the reserved word ! and command1 is a
subshell command, the application shall ensure that the ( operator at
the beginning of command1 is separated from the ! by one or more
<blank> characters. The behavior of the reserved word ! immediately
followed by the ( operator is unspecified.
So, just add a <blank> to prevent this.
[0] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_09_02
Signed-off-by: Michael Forney <mforney@mforney.org>
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions