Skip to content

bitcoin-core: fix seccp256k1 configure option#12299

Merged
DavidKorczynski merged 1 commit into
google:masterfrom
fanquake:secp_unrecognized_option
Jul 31, 2024
Merged

bitcoin-core: fix seccp256k1 configure option#12299
DavidKorczynski merged 1 commit into
google:masterfrom
fanquake:secp_unrecognized_option

Conversation

@fanquake
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

cc @sipa @real-or-random

@github-actions
Copy link
Copy Markdown

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
@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