Skip to content
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

docker: install clang-tools-18 #2058

Merged
merged 1 commit into from
Feb 11, 2024
Merged

Conversation

tchaikov
Copy link
Contributor

to address the build failure caused by
llvm/llvm-project#59827, and to test the build with C++23, we need to use clang-18. which will be released in March 2023.

@tchaikov
Copy link
Contributor Author

tested using

$ podman build -t seastar-dev -f ./docker/dev/Dockerfile  .
...
STEP 10/10: CMD /bin/bash
COMMIT seastar-dev
--> c4401e6db9d7
Successfully tagged localhost/seastar-dev:latest
c4401e6db9d7057e6ab910c49a1a2b2bdee5239953c012748cb3406a898739c8

@tchaikov
Copy link
Contributor Author

@avikivity could you help review this change?

1 similar comment
@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 1, 2024

@avikivity could you help review this change?

@avikivity
Copy link
Member

Why install an unreleased compiler? Is the fix not backported to clang 17?

@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 1, 2024

Why install an unreleased compiler? Is the fix not backported to clang 17?

no, clang-17 won't get any backport anymore. i prepared the backport on f38, f39 and rawhide, see https://src.fedoraproject.org/rpms/clang/pull-request/226 and https://src.fedoraproject.org/rpms/clang/pull-request/225. but i am too lazy to file a ticket on BTS and/ore on launchpad.

@avikivity
Copy link
Member

ok. It's not great but not much we can do.

avikivity pushed a commit to avikivity/seastar that referenced this pull request Feb 1, 2024
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
tchaikov added a commit to tchaikov/seastar that referenced this pull request Feb 1, 2024
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

since we now install `clang-tools-18`, circleci is updated to
build C++20 module with clang-18. as clang-tools-18 provides
clang-scan-deps.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
tchaikov added a commit to tchaikov/seastar that referenced this pull request Feb 1, 2024
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

since we now install `clang-tools-18`, circleci is updated to
build C++20 module with clang-18. as clang-tools-18 provides
clang-scan-deps.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
@tchaikov tchaikov force-pushed the clang-18 branch 2 times, most recently from 100ffd9 to 79cda5e Compare February 1, 2024 16:50
tchaikov added a commit to tchaikov/seastar that referenced this pull request Feb 1, 2024
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

since we now install `clang-tools-18`, circleci is updated to
build C++20 module with clang-18. as clang-tools-18 provides
clang-scan-deps.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

since we now install `clang-tools-18`, circleci is updated to
build C++20 module with clang-18. as clang-tools-18 provides
clang-scan-deps.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 5, 2024

v2:

  • rebased against master HEAD

@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 5, 2024

@avikivity hi Avi, mind taking another look?

@tchaikov
Copy link
Contributor Author

@avikivity ping?

@avikivity
Copy link
Member

Is this the same build script as was used to generate the image referenced in .circleci? I don't remember the gpg stuff.

@tchaikov
Copy link
Contributor Author

Is this the same build script as was used to generate the image referenced in .circleci? I don't remember the gpg stuff.

@avikivity it is. please compare it with #2070 . without installing GnuPG, it's impossible to import LLVM's keyring into the system, and we would not be able to install their debian packages without bypassing the signature check.

@avikivity avikivity merged commit 233b48c into scylladb:master Feb 11, 2024
14 checks passed
@tchaikov tchaikov deleted the clang-18 branch February 11, 2024 14:38
graphcareful pushed a commit to graphcareful/seastar that referenced this pull request Mar 20, 2024
to address the build failure caused by
llvm/llvm-project#59827, and to test
the build with C++23, we need to use clang-18. which will be
released in March 2023.

since we now install `clang-tools-18`, circleci is updated to
build C++20 module with clang-18. as clang-tools-18 provides
clang-scan-deps.

Signed-off-by: Kefu Chai <[email protected]>

Closes scylladb#2058

[avi: regenerate toolchain]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants