Re: [PATCH] selftests: futex: fix compilation error

From: Shuah Khan
Date: Wed Nov 08 2017 - 10:32:05 EST


On 11/08/2017 12:55 AM, lei yang wrote:
>
>
> On 2017å11æ08æ 06:57, Shuah Khan wrote:
>> On 11/06/2017 06:18 PM, lei yang wrote:
>>>
>>> On 2017å11æ07æ 07:48, Shuah Khan wrote:
>>>> On 11/06/2017 04:45 PM, Shuah Khan wrote:
>>>>> On 11/05/2017 09:03 PM, Lei Yang wrote:
>>>>>> I run into below error when building futext
>>>>>> /bin/sh: -c: line 5: syntax error: unexpected end of file
>>>>>>
>>>>>> the closing ";" and "\" are necessary.
>>>>>> My OS is "Ubuntu 14.04.5 LTS"
>>>>>>
>>>>>> Signed-off-by: Lei Yang <Lei.Yang@xxxxxxxxxxxxx>
>>>>>> ---
>>>>>> ÂÂ tools/testing/selftests/futex/Makefile | 6 +++---
>>>>>> ÂÂ 1 file changed, 3 insertions(+), 3 deletions(-)
>>>>>>
>>>>>> diff --git a/tools/testing/selftests/futex/Makefile b/tools/testing/selftests/futex/Makefile
>>>>>> index f0c0369..943f3a2 100644
>>>>>> --- a/tools/testing/selftests/futex/Makefile
>>>>>> +++ b/tools/testing/selftests/futex/Makefile
>>>>>> @@ -11,9 +11,9 @@ all:
>>>>>> ÂÂÂÂÂÂÂÂÂÂ BUILD_TARGET=$(OUTPUT)/$$DIR;ÂÂÂ \
>>>>>> ÂÂÂÂÂÂÂÂÂÂ mkdir $$BUILD_TARGETÂ -p;ÂÂÂ \
>>>>>> ÂÂÂÂÂÂÂÂÂÂ make OUTPUT=$$BUILD_TARGET -C $$DIR $@;\
>>>>>> -ÂÂÂÂÂÂÂ if [ -e $$DIR/$(TEST_PROGS) ]; then
>>>>>> -ÂÂÂÂÂÂÂÂÂÂÂ rsync -a $$DIR/$(TEST_PROGS) $$BUILD_TARGET/;
>>>>>> -ÂÂÂÂÂÂÂ fi
>>>>>> +ÂÂÂÂÂÂÂ if [ -e $$DIR/$(TEST_PROGS) ]; then \
>>>>>> +ÂÂÂÂÂÂÂÂÂÂÂ rsync -a $$DIR/$(TEST_PROGS) $$BUILD_TARGET/; \
>>>>>> +ÂÂÂÂÂÂÂ fi \
>>>>>> ÂÂÂÂÂÂ done
>>>>>> ÂÂÂÂ override define RUN_TESTS
>>>>>>
>>>>> Hmm. I don't see this error on linux_4.14-rc8?
>>>>>
>>>>> Can you give more details on the gcc version you are using?
>>>> I meant gnumake version? I am not seeing the failure with what I am
>>>> using
>>> $ make -v
>>> GNU Make 3.81
>>> Copyright (C) 2006Â Free Software Foundation, Inc.
>>> This is free software; see the source for copying conditions.
>>> There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
>>> PARTICULAR PURPOSE.
>>>
>>> This program built for x86_64-pc-linux-gnu
>>>
>>>
>>> Lei
>>>
>> I can't take patches that aren't tested on the latest kernel release.
>> 3.16 is very old. I am not going reply to other patches you sent.
>>
>> Please make sure the patches are based on the latest release and tested
>> on it as well.
>
> this error only happens when use old version "make", it has nothing to do with the new or old kernel.
>
> so the question is do we need to be compatible with old "make" ?
>
> Lei
>

Like I said, Linux 3.16 kernel is very old and I won't take patches that
are tested on it.

thanks,
-- Shuah