Hennadii Stepanov
Hennadii Stepanov
When building with clang and the `-Wshorten-64-to-32` flag, there are some warnings: ``` In file included from /home/hebasto/git/secp256k1/secp256k1/src/secp256k1.c:27: In file included from /home/hebasto/git/secp256k1/secp256k1/src/field_impl.h:14: In file included from /home/hebasto/git/secp256k1/secp256k1/src/field_5x52_impl.h:13: /home/hebasto/git/secp256k1/secp256k1/src/modinv64_impl.h:279:39: warning:...
During the integration of libsecp256k1's build system into Bitcoin Core's build system, it was decided to always build the most tested "RelWithDebInfo" configuration, regardless of the Bitcoin Core's actual build...
The "experimental" status of the CMake-based build system can be reconsidered once it is [adopted](https://github.com/bitcoin/bitcoin/pull/30454) in the Bitcoin Core project. Fixing all known bugs is essential before this reconsideration. Currently,...
This change: 1. Makes the `libqrencode.pc` files generated by Autotools and CMake more aligned. 2. Allows the `prefix` variable to be redefined if the package is relocated. For example: ```...
This PR sets the target output locations to the `bin` and `lib` subdirectories within the build tree, creating a directory structure that mirrors that of the installed targets. This approach...
On the master branch @ 9a7206a34e3179d7280de98a818a13374178ee7b, linking `bitcoin-chainstate.exe` to `bitcoinkernel.dll` fails: ``` > cmake -B build --preset vs2022-static -DBUILD_UTIL_CHAINSTATE=ON -DBUILD_KERNEL_LIB=ON -DBUILD_SHARED_LIBS=ON > cmake --build build --config Release -t bitcoin-chainstate ......
This PR switches to the latest [IWYU 0.23](https://github.com/include-what-you-use/include-what-you-use/releases/tag/0.23), which is compatible with Clang 19. New "bugprone-use-after-move" and "modernize-use-starts-ends-with" warnings that emerged have been addressed.
This PR enables on the CI tests of cross-compiled Windows binaries on Windows. It is important to have such tests in CI because the release binaries for Windows are also...
This change is required to pass cross builds on to a different machine after the build. For tests managed by CTest, the new `UNIVALUE_JSON_TEST_SRC` environment variable is set automatically. For...
This PR is the first step towards treating IWYU warnings as errors. It utilises CMake's native support via the [`CMAKE_CXX_INCLUDE_WHAT_YOU_USE`](https://cmake.org/cmake/help/latest/variable/CMAKE_LANG_INCLUDE_WHAT_YOU_USE.html)) variable. While this approach introduces some duplication at present, it...