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
tidy: remove todo, set minimum CMake to 3.22 #29696
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. |
Concept ACK, obviously :) |
Matches hebasto#123. This also also dev/ci only code.
e1d29dd
to
cf98578
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK cf98578.
It seems, the zstd
package might be ignored as well:
set(CMAKE_DISABLE_FIND_PACKAGE_zstd ON)
UPD: ... and LibEdit
?
At the same time, also disable searching for CURL, LibEdit, LibXml2, ZLIB and zstd none of which we use.
cf98578
to
5b690ae
Compare
Added LibXml2 while we are at it. |
Maybe also disable unneeded C compiler check: --- a/contrib/devtools/bitcoin-tidy/CMakeLists.txt
+++ b/contrib/devtools/bitcoin-tidy/CMakeLists.txt
@@ -1,6 +1,10 @@
cmake_minimum_required(VERSION 3.9)
-project(bitcoin-tidy VERSION 1.0.0 DESCRIPTION "clang-tidy checks for Bitcoin Core")
+project(bitcoin-tidy
+ VERSION 1.0.0
+ DESCRIPTION "clang-tidy checks for Bitcoin Core"
+ LANGUAGES CXX
+)
include(GNUInstallDirs)
? |
Also requires disabling FFI.
That works, but only after also disabling FFI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
re-ACK 11ee058.
See hebasto#123 for the minimum version bump.