-
-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding Clang 17 #28
Comments
Thanks for the letting me know. Most of the images should now be available. Note that the |
I cannot reproduce the issue of broken dependencies in a clean docker container. Ran the following commands locally without any issues: $ docker run --rm -it debian:bookworm-slim
# inside the container:
$ apt-get -qq update; \
apt-get install -qqy --no-install-recommends \
gnupg2 wget ca-certificates apt-transport-https \
autoconf automake cmake dpkg-dev file make patch libc6-dev
$ echo "deb https://apt.llvm.org/bookworm llvm-toolchain-bookworm-17 main" > /etc/apt/sources.list.d/llvm.list && wget -qO /etc/apt/trusted.gpg.d/llvm.asc https://apt.llvm.org/llvm-snapshot.gpg.key && apt-get -qq update && apt-get install -qqy -t llvm-toolchain-bookworm-17 clang-17 clang-tidy-17 clang-format-17 lld-17 libc++-17-dev libc++abi-17-dev
$ /usr/lib/llvm-17/bin/clang --version |
@jdoubleu are you running that on an arm64 or s390x platform? (amd64 does work for some reason) |
Sorry, misread the arm64. I was running the experiment successfully on amd64. Unfortunately, I cannot pinpoint the exact issue from the build logs, but was able to get more information, when trying to install the arm64 packages using multiarch:
Looks like an issue with the clang apt repository. Would it still be possible to just upload the amd64 image? You're building all architecture (images) in one job, maybe you can spread the work using matrix builds (or just generating more fine-grained jobs). Then, arm64 and s390x can fail, while amd64 would be unaffected. Would this be an option? I can help contribute that. |
This always trips me up as well 😓
Not sure if I currently have sufficiently fine-grained control over that with the current scripts, but this should be doable.
The last time I did some work on this it seemed like the image needed to be pushed for all architectures at the same time. Not sure if I messed something up though. Separate builds would definitely be a welcome improvement. |
I just noticed that the builds for clang 17 on bookworm (and buster) succeeded, after I've forked your repo. Could you maybe re-run the failed jobs/pipeline?
I'll have a look into the build system and try to come up with a solution, when I find the time. |
Hey, it'd be great to add Clang 17 as Clang 17 is now available; https://github.com/llvm/llvm-project/releases/tag/llvmorg-17.0.3
The text was updated successfully, but these errors were encountered: