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

bitcoin-core: fix seccp256k1 configure option #12299

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

fanquake
Copy link
Contributor

This option was renamed in bitcoin-core/secp256k1#1058, resulting in configure: WARNING: unrecognized options: --with-ecmult-gen-precision.

Update the option and change the values.

Also add --enable-option-checking, so that it'll be harder for this to happen in future. See:
https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.72/html_node/Option-Checking.html.

This option was renamed in
bitcoin-core/secp256k1#1058, resulting in
`configure: WARNING: unrecognized options: --with-ecmult-gen-precision`.

Update the option and change the values.

Also add `--enable-option-checking`, so that it'll be harder for this to
happen in future. See:
https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.72/html_node/Option-Checking.html.
@fanquake
Copy link
Contributor Author

cc @sipa @real-or-random

Copy link

fanquake (verified) is either the primary contact or is in the CCs list of projects/bitcoin-core.

@DavidKorczynski DavidKorczynski merged commit c383795 into google:master Jul 31, 2024
16 checks passed
@fanquake fanquake deleted the secp_unrecognized_option branch August 1, 2024 09:33
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.

2 participants