Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Sebastian Weyer <sebastian.weyer@smile.fr>
Cc: Lang Daniel <d.lang@abatec.at>,
	Romain Naour <romain.naour@smile.fr>,
	Adam Duskett <aduskett@gmail.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v4 00/10] group llvm projects and bump to version 15.0.3
Date: Thu, 20 Jul 2023 23:05:29 +0200	[thread overview]
Message-ID: <20230720230529.7c0142fd@windsurf> (raw)
In-Reply-To: <20230615134315.494330-1-sebastian.weyer@smile.fr>

Hello Sebastian,

On Thu, 15 Jun 2023 15:43:04 +0200
Sebastian Weyer <sebastian.weyer@smile.fr> wrote:

> Lang Daniel (10):
>   package/llvm-project: new group for llvm packages
>   package/llvm: move to llvm-project folder
>   package/lld: move to llvm-project subfolder
>   package/clang: move to llvm-project subfolder
>   package/compiler-rt: move to llvm-project subfolder
>   package/libclc: move to llvm-project subfolder
>   package/llvm-project/llvm-cmake: new package
>   package/llvm-project/llvm-libunwind: new package
>   package/llvm-project: bump to version 15.0.3
>   DEVELOPERS: add Daniel Lang for llvm-project

Thanks a lot, I have applied the entire patch series, with some fairly
small changes:

- I have squashed the patch touching the DEVELOPERS file into the first
  patch adding package/llvm-project in the first place. This ensures
  this directory isn't introduced as an "orphan" directory.

- I have taken care of updating the .checkpackageignore file throughout
  the patch series, which was not taken into account here.

- The final patch doing the big bump forgot the case of
  BR2_PACKAGE_QT5TOOLS_QDOC_TOOL which has a dependency on host-clang
  and therefore now depends on host gcc >= 7. And I see I forgot to
  also update the dependency on gcc >= 7. I'll fix that up.

  In fact, as noted in the definition of this option, there is still
  something wrong here: this qt5tools option only needs host-clang, but
  for some reason, we can only build host-clang when the target
  architecture is supported, which doesn't make much sense. It would be
  nice to look into this.

Thanks a lot for your efforts on this, and sorry for the long delay to
review and merge this.

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  parent reply	other threads:[~2023-07-20 22:07 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-15 13:43 [Buildroot] [PATCH v4 00/10] group llvm projects and bump to version 15.0.3 Sebastian Weyer
2023-06-15 13:43 ` [Buildroot] [PATCH v4 01/10] package/llvm-project: new group for llvm packages Sebastian Weyer
2023-07-20 17:30   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 02/10] package/llvm: move to llvm-project folder Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 03/10] package/lld: move to llvm-project subfolder Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 04/10] package/clang: " Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 05/10] package/compiler-rt: " Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 06/10] package/libclc: " Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 07/10] package/llvm-project/llvm-cmake: new package Sebastian Weyer
2023-07-20 17:31   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 08/10] package/llvm-project/llvm-libunwind: " Sebastian Weyer
2023-07-20 17:32   ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 09/10] package/llvm-project: bump to version 15.0.3 Sebastian Weyer
2023-07-12 13:06   ` Romain Naour
2023-07-20 17:36     ` Adam Duskett
2023-06-15 13:43 ` [Buildroot] [PATCH v4 10/10] DEVELOPERS: add Daniel Lang for llvm-project Sebastian Weyer
2023-07-20 17:32   ` Adam Duskett
2023-07-18  0:01 ` [Buildroot] [PATCH v4 00/10] group llvm projects and bump to version 15.0.3 Adam Duskett
2023-07-20 21:05 ` Thomas Petazzoni via buildroot [this message]
     [not found] <20230615134315.494330-1-sebastian.weyer__49449.699586124$1686836642$gmane$org@smile.fr>
2023-07-20 19:00 ` Bernd Kuhls
2023-07-20 21:06   ` Thomas Petazzoni via buildroot

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=20230720230529.7c0142fd@windsurf \
    --to=buildroot@buildroot.org \
    --cc=aduskett@gmail.com \
    --cc=d.lang@abatec.at \
    --cc=romain.naour@smile.fr \
    --cc=sebastian.weyer@smile.fr \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox