build: Bump `native_clang` up to 17.0.2

hebasto commented at 7:20 pm on October 25, 2023: member

This PR makes it possible to build the qt package with macOS 14 SDK (Xcode 15.0).

For more details, please refer to #28622.

build: Bump `native_clang` up to 17.0.2 b3c8caf51f

DrahtBot commented at 7:20 pm on October 25, 2023: contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type	Reviewers
Concept NACK	fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

DrahtBot added the label Build system on Oct 25, 2023

maflcko added the label DrahtBot Guix build requested on Oct 25, 2023

fanquake commented at 7:30 pm on October 25, 2023: member

I don’t understand how bumping Clang would fix building with a newer SDK? How is this related?

Concept NACK in any case, at least until we have LLVM 17 in Guix.

maflcko commented at 7:36 pm on October 25, 2023: member

Yes, would be good to keep guix in sync. Otherwise the CI will be less useful. See also #28337 and #28580 (comment)

DrahtBot commented at 1:15 am on October 26, 2023: contributor

Guix builds (on x86_64)

File	commit 2a349f9ea5de9f0157cd117289996e8640473a6d(master)	commit cf2e4f57fdbe2da9157df79ca40047cd9b12680b(master and this pull)
SHA256SUMS.part	`caad4c31d5f25a98...`	`f35c8fca3463e4ee...`
*-aarch64-linux-gnu-debug.tar.gz	`3ea054e6597c0dc6...`	`501f6b23365a6ad2...`
*-aarch64-linux-gnu.tar.gz	`07c63f58b61c1540...`	`3d4588c1633d77cd...`
*-arm-linux-gnueabihf-debug.tar.gz	`045d603b322e9882...`	`31b44731505e158c...`
*-arm-linux-gnueabihf.tar.gz	`aa6cee3feb14e3c2...`	`33b1372cb094cb78...`
*-arm64-apple-darwin-unsigned.tar.gz	`5bedae6a3c6ff1b2...`	`eccaa0bc9be32752...`
*-arm64-apple-darwin-unsigned.zip	`537b16729842ae84...`	`a1d713af1e0bf365...`
*-arm64-apple-darwin.tar.gz	`372165e3cefb7c1d...`	`c1f1cddf36ea5312...`
*-powerpc64-linux-gnu-debug.tar.gz	`57f3d71d6420f425...`	`f4bdac5bafe7d69e...`
*-powerpc64-linux-gnu.tar.gz	`f75fd0ed267ff7e7...`	`f93fb7faba44e68c...`
*-powerpc64le-linux-gnu-debug.tar.gz	`2a7002fa719a8ccc...`	`5553845b30151074...`
*-powerpc64le-linux-gnu.tar.gz	`d949e85ff3ac4494...`	`370f5d9087cd253f...`
*-riscv64-linux-gnu-debug.tar.gz	`bc68d3e9d99d122d...`	`c31a94af6c67aa9b...`
*-riscv64-linux-gnu.tar.gz	`0041c26cc04243c8...`	`b72d1cf7d751635b...`
*-x86_64-apple-darwin-unsigned.tar.gz	`95115701c06625f4...`	`e4c322980ad01f0d...`
*-x86_64-apple-darwin-unsigned.zip	`af69076558fa7b19...`	`dd36ba3cacbe92cd...`
*-x86_64-apple-darwin.tar.gz	`67aa4fa79118993c...`	`3f0f13f965e34435...`
*-x86_64-linux-gnu-debug.tar.gz	`0875eb48a365db43...`	`06030a5a30af8243...`
*-x86_64-linux-gnu.tar.gz	`e824a32f574eb0d9...`	`62bceaeaf2f25a64...`
*.tar.gz	`f5cced582675dd30...`	`eae26c1c6b67611c...`
guix_build.log	`e848507d89c4fa44...`	`38acd527aa5cf84f...`
guix_build.log.diff		`82be11389f1e4af7...`

DrahtBot removed the label DrahtBot Guix build requested on Oct 26, 2023

fanquake commented at 12:31 pm on October 26, 2023: member

Also, if Clang 17 is required, to compile with the Xcode 15 SDK, wouldn’t that make it impossible to compile on macOS using Apple Clang, given the most recent version is only based on LLVM 16?

hebasto commented at 12:39 pm on October 26, 2023: member

Also, if Clang 17 is required, to compile with the Xcode 15 SDK, wouldn’t that make it impossible to compile on macOS using Apple Clang, given the most recent version is only based on LLVM 16?

From my experience, Clang 16 causes the same errors as the current Clang 15.0.6 does.

fanquake commented at 12:46 pm on October 26, 2023: member

Sure, but I can currently use an Apple Clang 15.0.0 (based on LLVM 16), with the Xcode 15 SDK, and everything works ok. I still don’t understand what the actual problem is, what has supposedly changed in Clang 17 that fixes it, or how Apple would have managed to change the SDK in such a way that it only works with the most recently released version of Clang.

fanquake commented at 1:39 pm on October 31, 2023: member

Can you explain this change? Using a newer Xcode is one of the last the blockers for C++20.

fanquake marked this as a draft on Oct 31, 2023

fanquake commented at 1:58 pm on October 31, 2023: member

Discussed further offline. Moving this to draft for now. As mentioned, bumping Clang isn’t required to use C++20, because bitcoind and other utils can already be compiled using C++20, against the newer SDK, so the assumption is that this is something that will need to be patched in Qt?

hebasto commented at 1:45 pm on November 2, 2023: member

Closing in favour of #28775.

hebasto closed this on Nov 2, 2023

build: Bump native_clang up to 17.0.2 #28732

Code Coverage

Reviews

Guix builds (on x86_64)

build: Bump `native_clang` up to 17.0.2 #28732