diff mbox series

[v2] selftests/clone3: Fix build error

Message ID 1606447017-10640-1-git-send-email-suxingxing@loongson.cn
State Accepted
Commit 88f4ede44c585b24674dd99841040b2a1a856a76
Headers show
Series [v2] selftests/clone3: Fix build error | expand

Commit Message

suxingxing Nov. 27, 2020, 3:16 a.m. UTC
When compiling the selftests with the -std=gnu99 option the build can
fail with.

Following build error:

  test_core.c: In function ‘test_cgcore_destroy’:
  test_core.c:87:2: error: ‘for’ loop initial declarations are only
  allowed in C99 mode
    for (int i = 0; i < 10; i++) {
    ^
  test_core.c:87:2: note: use option -std=c99 or -std=gnu99 to compile

Add -std=gnu99 to the clone3 selftest Makefile to fix this.

Signed-off-by: Xingxing Su <suxingxing@loongson.cn>
Acked-by: Christian Brauner <christian.brauner@ubuntu.com>
---
 tools/testing/selftests/clone3/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Shuah Khan Dec. 7, 2020, 10:02 p.m. UTC | #1
On 11/26/20 8:16 PM, Xingxing Su wrote:
> When compiling the selftests with the -std=gnu99 option the build can

> fail with.

> 

> Following build error:

> 

>    test_core.c: In function ‘test_cgcore_destroy’:

>    test_core.c:87:2: error: ‘for’ loop initial declarations are only

>    allowed in C99 mode

>      for (int i = 0; i < 10; i++) {

>      ^

>    test_core.c:87:2: note: use option -std=c99 or -std=gnu99 to compile

> 

> Add -std=gnu99 to the clone3 selftest Makefile to fix this.

> 

> Signed-off-by: Xingxing Su <suxingxing@loongson.cn>

> Acked-by: Christian Brauner <christian.brauner@ubuntu.com>

> ---

>   tools/testing/selftests/clone3/Makefile | 2 +-

>   1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/tools/testing/selftests/clone3/Makefile b/tools/testing/selftests/clone3/Makefile

> index ef7564c..88354a8 100644

> --- a/tools/testing/selftests/clone3/Makefile

> +++ b/tools/testing/selftests/clone3/Makefile

> @@ -1,5 +1,5 @@

>   # SPDX-License-Identifier: GPL-2.0

> -CFLAGS += -g -I../../../../usr/include/

> +CFLAGS += -g -std=gnu99 -I../../../../usr/include/

>   LDLIBS += -lcap

>   

>   TEST_GEN_PROGS := clone3 clone3_clear_sighand clone3_set_tid \

> 


Applied to linux-kselftest fixes for 5.11-rc1

thanks,
-- Shuah
diff mbox series

Patch

diff --git a/tools/testing/selftests/clone3/Makefile b/tools/testing/selftests/clone3/Makefile
index ef7564c..88354a8 100644
--- a/tools/testing/selftests/clone3/Makefile
+++ b/tools/testing/selftests/clone3/Makefile
@@ -1,5 +1,5 @@ 
 # SPDX-License-Identifier: GPL-2.0
-CFLAGS += -g -I../../../../usr/include/
+CFLAGS += -g -std=gnu99 -I../../../../usr/include/ 
 LDLIBS += -lcap
 
 TEST_GEN_PROGS := clone3 clone3_clear_sighand clone3_set_tid \