From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Cc: Grzegorz Blach <grzegorz@blach.pl>,
Anisse Astier <anisse@astier.eu>, Asaf Kahlon <asafka7@gmail.com>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
buildroot@buildroot.org, Adam Duskett <aduskett@gmail.com>
Subject: Re: [Buildroot] [PATCH 1/1] package/gobject-introspection: needs host gcc >= 7
Date: Mon, 25 Oct 2021 22:18:05 +0200 [thread overview]
Message-ID: <20211025201805.GT2400@scaer> (raw)
In-Reply-To: <20211019052531.1604264-1-fontaine.fabrice@gmail.com>
Fabrice, All,
+Adam
On 2021-10-19 07:25 +0200, Fabrice Fontaine spake thusly:
> Commit 33c69c02fb4c1176584d1e2dd336b74ed39f5d65 forgot to propagate
> host gcc >= 7 dependency to reverse dependencies of host-qemu resulting
> in the following build failure:
>
> ERROR: You need at least GCC v7.5 or Clang v6.0 (or XCode Clang v10.0)
>
> Fixes:
> - http://autobuild.buildroot.org/results/14cd0250d55c5ca5dc1d297aee1b927fda889531
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Applied to master, thanks.
Note however, that the issue is really about gcc >= 7.5. We do not have
such a symbol, and we do not want to add it. So, adding the dependency
on host gcc >= 7 is not strictly correct, because it allows for gcc 7.4
and below... And gcc 7.5 was the last release in the 7.x branch. It is
probably a safe bet to require 8.x now.
Adam has sent a patch doing that; I already asked him on IRC to rebase
on master now that I applied your patch.
Regards,
Yann E. MORIN.
> ---
> package/gobject-introspection/Config.in | 6 ++++--
> package/python-bluezero/Config.in | 6 ++++--
> package/python-gobject/Config.in | 6 ++++--
> 3 files changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/package/gobject-introspection/Config.in b/package/gobject-introspection/Config.in
> index bacf44363c..c631c84755 100644
> --- a/package/gobject-introspection/Config.in
> +++ b/package/gobject-introspection/Config.in
> @@ -13,6 +13,7 @@ config BR2_PACKAGE_GOBJECT_INTROSPECTION
> # compile errors with locale.c. As such, require 4.9 until this
> # can be verified.
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
> + depends on BR2_HOST_GCC_AT_LEAST_7 # host-qemu
> depends on BR2_TOOLCHAIN_USES_GLIBC
> # gobject-introspection does not require python3 to run on the
> # target; however, because the tools run in a qemu wrapper, a
> @@ -41,8 +42,9 @@ comment "gobject-introspection needs python3"
> depends on !BR2_PACKAGE_PYTHON3
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS
>
> -comment "gobject-introspection needs a glibc toolchain, gcc >= 4.9"
> +comment "gobject-introspection needs a glibc toolchain, gcc >= 4.9, host gcc >= 7"
> depends on BR2_USE_MMU
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS
> depends on !BR2_TOOLCHAIN_USES_GLIBC || \
> - !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
> + !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
> + !BR2_HOST_GCC_AT_LEAST_7
> diff --git a/package/python-bluezero/Config.in b/package/python-bluezero/Config.in
> index c391bfa858..3d40b1e275 100644
> --- a/package/python-bluezero/Config.in
> +++ b/package/python-bluezero/Config.in
> @@ -2,6 +2,7 @@ config BR2_PACKAGE_PYTHON_BLUEZERO
> bool "python-bluezero"
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS # python-gobject -> gobject-introspection
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # python-gobject -> gobject-introspection
> + depends on BR2_HOST_GCC_AT_LEAST_7 # python-gobject -> gobject-introspection -> host-qemu
> depends on BR2_TOOLCHAIN_USES_GLIBC # python-gobject -> gobject-introspection
> depends on BR2_PACKAGE_PYTHON3
> select BR2_PACKAGE_DBUS # runtime
> @@ -12,8 +13,9 @@ config BR2_PACKAGE_PYTHON_BLUEZERO
>
> https://github.com/ukBaz/python-bluezero
>
> -comment "python-bluezero needs a glibc toolchain, gcc >= 4.9"
> +comment "python-bluezero needs a glibc toolchain, gcc >= 4.9, host gcc >= 7"
> depends on BR2_PACKAGE_PYTHON3
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS
> depends on !BR2_TOOLCHAIN_USES_GLIBC || \
> - !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
> + !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
> + !BR2_HOST_GCC_AT_LEAST_7
> diff --git a/package/python-gobject/Config.in b/package/python-gobject/Config.in
> index 1c9303b0ba..910f990026 100644
> --- a/package/python-gobject/Config.in
> +++ b/package/python-gobject/Config.in
> @@ -3,6 +3,7 @@ config BR2_PACKAGE_PYTHON_GOBJECT
> depends on BR2_USE_MMU # libglib2, gobject-introspection
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS # gobject-introspection
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # gobject-introspection
> + depends on BR2_HOST_GCC_AT_LEAST_7 # gobject-introspection -> host-qemu
> depends on BR2_TOOLCHAIN_USES_GLIBC # gobject-introspection
> depends on BR2_PACKAGE_PYTHON3 # gobject-introspection
> select BR2_PACKAGE_GOBJECT_INTROSPECTION
> @@ -12,8 +13,9 @@ config BR2_PACKAGE_PYTHON_GOBJECT
>
> https://wiki.gnome.org/action/show/Projects/PyGObject
>
> -comment "python-gobject needs a glibc toolchain, gcc >= 4.9"
> +comment "python-gobject needs a glibc toolchain, gcc >= 4.9, host gcc >= 7"
> depends on BR2_USE_MMU
> depends on BR2_PACKAGE_GOBJECT_INTROSPECTION_ARCH_SUPPORTS
> depends on !BR2_TOOLCHAIN_USES_GLIBC || \
> - !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
> + !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
> + !BR2_HOST_GCC_AT_LEAST_7
> --
> 2.33.0
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2021-10-25 20:18 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-19 5:25 [Buildroot] [PATCH 1/1] package/gobject-introspection: needs host gcc >= 7 Fabrice Fontaine
2021-10-25 20:18 ` Yann E. MORIN [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211025201805.GT2400@scaer \
--to=yann.morin.1998@free.fr \
--cc=aduskett@gmail.com \
--cc=anisse@astier.eu \
--cc=asafka7@gmail.com \
--cc=buildroot@buildroot.org \
--cc=fontaine.fabrice@gmail.com \
--cc=grzegorz@blach.pl \
--cc=thomas.petazzoni@bootlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.