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

Switch to cpp-linter/clang-tools-static-binaries to download clang-tools #56

Closed
shenxianpeng opened this issue Jan 8, 2024 · 3 comments · Fixed by #57
Closed

Switch to cpp-linter/clang-tools-static-binaries to download clang-tools #56

shenxianpeng opened this issue Jan 8, 2024 · 3 comments · Fixed by #57
Assignees
Labels
enhancement New feature or request

Comments

@shenxianpeng
Copy link
Collaborator

shenxianpeng commented Jan 8, 2024

I think it's time to switch from muttleyxd/clang-tools-static-binaries to cpp-linter/clang-tools-static-binaries @2bndy5

We have known issues about clang-tidy-14, clang-tidy-15, and clang-tidy-16 having Segmentation faults on Ubuntu 22.02, but after we forked clang-tools-static-binaries repo, made some changes (dropped old version and upgrade build system version)rebuilt by ourselves, the Segmentation faults not existing anymore.

Here is my manual test on AlmaLinux release 8.9, our build clang-tidy-14 works.

# download from https://github.com/muttleyxd/clang-tools-static-binaries/releases/download/master-8f72ab3c/clang-tidy-14_linux-amd64 
bash-4.4$ ./clang-tidy-14_linux-amd64 --version
Segmentation fault (core dumped)

# download from https://github.com/cpp-linter/clang-tools-static-binaries/releases/download/master-be694ee7/clang-query-14_linux-amd64
bash-4.4$ ./clang-tidy-14_linux-amd64 --version
LLVM (http://llvm.org/):
  LLVM version 14.0.0
  Optimized build.
  Default target: x86_64-unknown-linux-gnu
  Host CPU: skylake-avx512
bash-4.4$

Please see more installation test on Linux, Windows, and MacOS from clang version 7 to 17

I don't exactly know the resean why this issue muttleyxd/clang-tools-static-binaries#18 was gone, the most likely reason is that we upgraded the Linux runner from the original ubuntu-20.04 to ubuntu-22.04.

@shenxianpeng shenxianpeng added the enhancement New feature or request label Jan 8, 2024
@shenxianpeng shenxianpeng changed the title Switch to cpp-linter/clang-tools-static-binaries download clang-tools Switch to cpp-linter/clang-tools-static-binaries to download clang-tools Jan 8, 2024
@2bndy5
Copy link
Contributor

2bndy5 commented Jan 8, 2024

the Segmentation faults not existing anymore

Awesome! I'm all in.

Please see more installation test on Linux, Windows, and MacOS from clang version 7 to 17

I see you tested all on x86_64 architectures. I'm curious if we should warn in the README about architectures that are not supported (like arm64 aka aarch64 and x86).

Maybe the badge is enough?

I don't exactly resean why this issue muttleyxd/clang-tools-static-binaries#18 was fixed, the most likely reason is that we upgraded our Linux build system from the original ubuntu-20.04 to ubuntu-22.04.

The error would still surface on Linux runners that use older versions of libc. At least that was my understanding.


Lastly, all clang-tidy v14+ builds for MacOS are still ~1.8 GB in size.

@shenxianpeng
Copy link
Collaborator Author

Lastly, all clang-tidy v14+ builds for MacOS are still ~1.8 GB in size.

This is still the same on MacOS

I see you tested all on x86_64 architectures. I'm curious if we should warn in the README about architectures that are not supported (like arm64 aka aarch64 and x86).

Good suggestion. and the badge you created looks great.

The error would still surface on Linux runners that use older versions of libc. At least that was my understanding.

Yes, it should be, maybe I can test on older versions to have a try.


I guess it's time for you to go to bed. : )

@2bndy5
Copy link
Contributor

2bndy5 commented Jan 8, 2024

I'm in bed now, but the keyboard tends to follow me around... 🤣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants