Messages in this thread | | | Subject | Re: [PATCH 2/2] selftests: sgx: Treat CC as one argument | From | Shuah Khan <> | Date | Thu, 10 Feb 2022 13:52:04 -0700 |
| |
On 2/10/22 12:06 PM, Muhammad Usama Anjum wrote: > CC can have multiple sub-strings like "ccache gcc". For check_cc.sh, > CC needs to be treated like one argument. Put double quotes around it to > make CC one string and hence one argument. > > Fixes: 2adcba79e69d ("selftests/x86: Add a selftest for SGX") > Reported-by: "kernelci.org bot" <bot@kernelci.org> > Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> > --- > tools/testing/selftests/sgx/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/sgx/Makefile b/tools/testing/selftests/sgx/Makefile > index 2956584e1e37f..75af864e07b65 100644 > --- a/tools/testing/selftests/sgx/Makefile > +++ b/tools/testing/selftests/sgx/Makefile > @@ -4,7 +4,7 @@ include ../lib.mk > > .PHONY: all clean > > -CAN_BUILD_X86_64 := $(shell ../x86/check_cc.sh $(CC) \ > +CAN_BUILD_X86_64 := $(shell ../x86/check_cc.sh "$(CC)" \ > ../x86/trivial_64bit_program.c) > > ifndef OBJCOPY >
The intent is testing if $CC is set. Does this change work when $CC is not set?
thanks, -- Shuah
| |