All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/harfbuzz: needs host gcc >= 4.8
@ 2019-09-21 18:23 Bernd Kuhls
  2019-09-21 19:11 ` Arnout Vandecappelle
  0 siblings, 1 reply; 2+ messages in thread
From: Bernd Kuhls @ 2019-09-21 18:23 UTC (permalink / raw)
  To: buildroot

Fixes:
http://autobuild.buildroot.net/results/c20/c209e1128cf4601e2f7dc131f82eec0b5df39f75/

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/harfbuzz/Config.in | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/harfbuzz/Config.in b/package/harfbuzz/Config.in
index e8f407bb8f..4fa2135966 100644
--- a/package/harfbuzz/Config.in
+++ b/package/harfbuzz/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_HARFBUZZ
 	bool "harfbuzz"
+	depends on BR2_HOST_GCC_AT_LEAST_4_8
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
@@ -14,4 +15,5 @@ config BR2_PACKAGE_HARFBUZZ
 comment "harfbuzz needs a toolchain w/ C++, gcc => 4.8"
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	depends on !BR2_INSTALL_LIBSTDCPP || \
+		!BR2_HOST_GCC_AT_LEAST_4_8 || \
 		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH 1/1] package/harfbuzz: needs host gcc >= 4.8
  2019-09-21 18:23 [Buildroot] [PATCH 1/1] package/harfbuzz: needs host gcc >= 4.8 Bernd Kuhls
@ 2019-09-21 19:11 ` Arnout Vandecappelle
  0 siblings, 0 replies; 2+ messages in thread
From: Arnout Vandecappelle @ 2019-09-21 19:11 UTC (permalink / raw)
  To: buildroot



On 21/09/2019 20:23, Bernd Kuhls wrote:
> Fixes:
> http://autobuild.buildroot.net/results/c20/c209e1128cf4601e2f7dc131f82eec0b5df39f75/

 No it doesn't...

> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/harfbuzz/Config.in | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/harfbuzz/Config.in b/package/harfbuzz/Config.in
> index e8f407bb8f..4fa2135966 100644
> --- a/package/harfbuzz/Config.in
> +++ b/package/harfbuzz/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_HARFBUZZ
>  	bool "harfbuzz"
> +	depends on BR2_HOST_GCC_AT_LEAST_4_8

 If a target package depends on a host compiler version, you really have to
explain why...

 In this case, it doesn't, actually. It is pango that depends on host-harfbuzz,
and *that* requires a host compiler >= 4.8.

 So, you have to add the dependency to pango, not harfbuzz. Which means
propagating it...

 Regards,
 Arnout

>  	depends on BR2_INSTALL_LIBSTDCPP
>  	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
> @@ -14,4 +15,5 @@ config BR2_PACKAGE_HARFBUZZ
>  comment "harfbuzz needs a toolchain w/ C++, gcc => 4.8"
>  	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	depends on !BR2_INSTALL_LIBSTDCPP || \
> +		!BR2_HOST_GCC_AT_LEAST_4_8 || \
>  		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_8
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-09-21 19:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-21 18:23 [Buildroot] [PATCH 1/1] package/harfbuzz: needs host gcc >= 4.8 Bernd Kuhls
2019-09-21 19:11 ` Arnout Vandecappelle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.