All of 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 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.