Messages in this thread | | | From | Masahiro Yamada <> | Date | Tue, 14 Aug 2018 11:46:45 +0900 | Subject | Re: [PATCH] Support mksh as /bin/sh. |
| |
2018-08-14 7:32 GMT+09:00 Kees Cook <keescook@chromium.org>: > On Mon, Aug 13, 2018 at 3:14 AM, <arekm@maven.pl> wrote: >> From: Arkadiusz Miśkiewicz <arekm@maven.pl> >> >> mksh needs space between ! and ( to work properly. Otherwise this >> happens: >> >> + make oldconfig >> scripts/kconfig/conf --oldconfig Kconfig >> ./scripts/clang-version.sh[18]: COPYING: not found >> printf: ‘__clang_major__’: expected a numeric value >> printf: ‘__clang_minor__’: expected a numeric value >> printf: ‘__clang_patchlevel__’: expected a numeric value >> init/Kconfig:24:warning: 'CLANG_VERSION': number is invalid >> >> Signed-off-by: Arkadiusz Miśkiewicz <arekm@maven.pl> > > Reviewed-by: Kees Cook <keescook@chromium.org> > > -Kees >
I could not find this patch in ML.
Arkadiusz,
Please resend this patch to
linux-kbuild@vger.kernel.org
>> --- >> scripts/clang-version.sh | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/scripts/clang-version.sh b/scripts/clang-version.sh >> index dbf0a31eb111..e65fbc3079d4 100755 >> --- a/scripts/clang-version.sh >> +++ b/scripts/clang-version.sh >> @@ -12,7 +12,7 @@ >> >> compiler="$*" >> >> -if !( $compiler --version | grep -q clang) ; then >> +if ! ( $compiler --version | grep -q clang) ; then >> echo 0 >> exit 1 >> fi >> -- >> 2.18.0 >> > > > > -- > Kees Cook > Pixel Security
-- Best Regards Masahiro Yamada
| |