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

ci: use clang-format from LLVM 16 #3696

Merged
merged 5 commits into from
Aug 1, 2023
Merged

ci: use clang-format from LLVM 16 #3696

merged 5 commits into from
Aug 1, 2023

Conversation

XVilka
Copy link
Member

@XVilka XVilka commented Jul 28, 2023

Your checklist for this pull request

  • I've read the guidelines for contributing to this repository
  • I made sure to follow the project's coding style
  • I've documented or updated the documentation of every function and struct this PR changes. If not so I've explained why.
  • I've added tests that prove my fix is effective or that my feature works (if possible)
  • I've updated the rizin book with the relevant information (if needed)

Detailed description

While there is an error on LLVM 15, try LLVM 16:

+ apt-get update
Hit:1 http://azure.archive.ubuntu.com/ubuntu jammy InRelease
Hit:2 http://azure.archive.ubuntu.com/ubuntu jammy-updates InRelease
Hit:3 http://azure.archive.ubuntu.com/ubuntu jammy-backports InRelease
Hit:4 http://azure.archive.ubuntu.com/ubuntu jammy-security InRelease
Hit:5 https://packages.microsoft.com/ubuntu/22.04/prod jammy InRelease
Hit:6 http://security.ubuntu.com/ubuntu jammy-security InRelease
Hit:7 http://archive.ubuntu.com/ubuntu jammy InRelease
Hit:8 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
Hit:10 https://ppa.launchpadcontent.net/ubuntu-toolchain-r/test/ubuntu jammy InRelease
Hit:9 https://apt.llvm.org/jammy llvm-toolchain-jammy-15 InRelease
Reading package lists...
+ PKG='clang-15 lldb-15 lld-15 clangd-15'
+ [[ 0 -eq 1 ]]
+ apt-get install -y clang-15 lldb-15 lld-15 clangd-15
Reading package lists...
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 python3-lldb-14 : Conflicts: python3-lldb-x.y
 python3-lldb-15 : Conflicts: python3-lldb-x.y
E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.
Error: Process completed with exit code 100.

Test plan

CI is green

@XVilka
Copy link
Member Author

XVilka commented Jul 28, 2023

Blocked on mainstream problem: llvm/llvm-project#64182

@ret2libc
Copy link
Member

I don't see this error locally :(

@XVilka
Copy link
Member Author

XVilka commented Jul 31, 2023

It's mainstream problem, see the link above

@wargio
Copy link
Member

wargio commented Jul 31, 2023

@wargio wargio requested a review from yossizap as a code owner July 31, 2023 12:59
@wargio wargio merged commit 3278ec7 into dev Aug 1, 2023
45 checks passed
@wargio wargio deleted the clang-format-16 branch August 1, 2023 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants