-
Notifications
You must be signed in to change notification settings - Fork 36.2k
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: Add missing -Wno-error=maybe-uninitialized to armhf task #30144
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. |
utACK fa73431 |
@@ -17,4 +17,4 @@ export RUN_FUNCTIONAL_TESTS=false | |||
export GOAL="install" | |||
# -Wno-psabi is to disable ABI warnings: "note: parameter passing for argument of type ... changed in GCC 7.1" | |||
# This could be removed once the ABI change warning does not show up by default | |||
export BITCOIN_CONFIG="--enable-reduce-exports CXXFLAGS=-Wno-psabi" | |||
export BITCOIN_CONFIG="--enable-reduce-exports CXXFLAGS='-Wno-psabi -Wno-error=maybe-uninitialized'" |
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.
Good to see. Might be worth adding a comment about why the warning is disabled here and in ci/test/00_setup_env_win64.sh
?
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.
All -Wno-error=
in ./ci/
are due to false positives. I may address this, if I have to re-push for other reasons.
lgtm |
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 fa73431 - many fixed with 13.x
Ported to the CMake-based build system in hebasto#207. |
4b9b5c5 fixup! ci: Migrate CI scripts to CMake (Hennadii Stepanov) Pull request description: This PR ports bitcoin#30144. Top commit has no ACKs. Tree-SHA512: 56f3f4fc7408b1fa9887e979bf15de5ea7e8fe56d6e5194bfeaa862c164bf8f4d9259b4ae0121af8f4f4dbd6a43f23e82948fa6bac21175a0ba365d0d47bfd03
This happens after bd597c3 in many pull requests as a silent merge conflict. For example: