diff mbox series

[1/1] package/fio: needs gcc >= 4.9

Message ID 20210801201739.4190294-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/fio: needs gcc >= 4.9 | expand

Commit Message

Fabrice Fontaine Aug. 1, 2021, 8:17 p.m. UTC
fio needs gcc >= 4.9 since version 3.22 and
https://github.com/axboe/fio/commit/bb012314f734b9de3b2fc198db2c4e3e060db240
as a result build fails since bump to version 3.27 in commit
5c788892102f0e22c9b5694facca0576b09f5ef4

Fixes:
 - http://autobuild.buildroot.org/results/01a1e2f4c02e86f3c5b0eb333ed5f4dd6568edf8

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/fio/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Yann E. MORIN Aug. 1, 2021, 8:30 p.m. UTC | #1
Fabrice, All,

On 2021-08-01 22:17 +0200, Fabrice Fontaine spake thusly:
> fio needs gcc >= 4.9 since version 3.22 and
> https://github.com/axboe/fio/commit/bb012314f734b9de3b2fc198db2c4e3e060db240
> as a result build fails since bump to version 3.27 in commit
> 5c788892102f0e22c9b5694facca0576b09f5ef4
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/01a1e2f4c02e86f3c5b0eb333ed5f4dd6568edf8
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  package/fio/Config.in | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/package/fio/Config.in b/package/fio/Config.in
> index 8da227db7c..7825209b4c 100644
> --- a/package/fio/Config.in
> +++ b/package/fio/Config.in
> @@ -2,6 +2,7 @@ config BR2_PACKAGE_FIO
>  	bool "fio"
>  	depends on BR2_USE_MMU # fork()
>  	depends on !BR2_STATIC_LIBS
> +	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C11/stdatomic.h
>  	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	# fio uses fallocate() which becomes fallocate64() while compiling with
> @@ -13,8 +14,9 @@ config BR2_PACKAGE_FIO
>  
>  	  http://git.kernel.dk/?p=fio.git;a=summary
>  
> -comment "fio needs a toolchain w/ dynamic library, threads"
> -	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
> +comment "fio needs a toolchain w/ dynamic library, threads, gcc >= 4.9"
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS || \
> +		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
>  	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	depends on !BR2_nios2
>  	depends on BR2_USE_MMU
> -- 
> 2.30.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox series

Patch

diff --git a/package/fio/Config.in b/package/fio/Config.in
index 8da227db7c..7825209b4c 100644
--- a/package/fio/Config.in
+++ b/package/fio/Config.in
@@ -2,6 +2,7 @@  config BR2_PACKAGE_FIO
 	bool "fio"
 	depends on BR2_USE_MMU # fork()
 	depends on !BR2_STATIC_LIBS
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C11/stdatomic.h
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	# fio uses fallocate() which becomes fallocate64() while compiling with
@@ -13,8 +14,9 @@  config BR2_PACKAGE_FIO
 
 	  http://git.kernel.dk/?p=fio.git;a=summary
 
-comment "fio needs a toolchain w/ dynamic library, threads"
-	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
+comment "fio needs a toolchain w/ dynamic library, threads, gcc >= 4.9"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS || \
+		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	depends on !BR2_nios2
 	depends on BR2_USE_MMU