linux-kernel - Re: [PATCH] selftests: futex: fix compilation error

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f8400076-9971-8108-1d0d-25280a7025d3@windriver.com>
Date:   Wed, 8 Nov 2017 15:55:02 +0800
From:   lei yang <lei.yang@...driver.com>
To:     Shuah Khan <shuahkh@....samsung.com>
CC:     <linux-kselftest@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        Shuah Khan <shuah@...nel.org>
Subject: Re: [PATCH] selftests: futex: fix compilation error



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@...driver.com>
>>>>> ---
>>>>>    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



>
> thanks,
> -- Shuah
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html