Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented May 5, 2023

libsecp256k1 v0.3.0:

Removed the configuration header src/libsecp256k1-config.h.

This PR removed the code that has been unused since #27230.

The USE_ASM_X86_64 is now undefined explicitly (but actually it seems a bit redundant).

The ECMULT_GEN_PREC_BITS and ECMULT_WINDOW_SIZE macros are defined by the source code to their defaults.


Considering the upcoming CMake-based build system, these changes have a low-priority.

@DrahtBot
Copy link
Contributor

DrahtBot commented May 5, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #27479 (BIP324: ElligatorSwift integrations by sipa)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d9b54c4

@fanquake
Copy link
Member

fanquake commented May 6, 2023

Note that this conflicts with #27479, however it's a single-line change, and 27479 will have to be rebased again after the secp changes are pulled into our tree, so I think we can go ahead and merge this in the interim. cc @sipa.

@fanquake fanquake merged commit 26cb32c into bitcoin:master May 8, 2023
@pinheadmz
Copy link
Member

ACK
I noticed when switching locally between old and new branches that some untracked files got left over:


--> git status
On branch master
Your branch is up to date with 'origin/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	src/secp256k1/src/libsecp256k1-config.h
	src/secp256k1/src/libsecp256k1-config.h.in
	src/secp256k1/src/stamp-h1

nothing added to commit but untracked files present (use "git add" to track)

@pinheadmz
Copy link
Member

Actually I still get the untracked files warning after merge -- am i doing something wrong? Or would it make any sense to add these to gitignore?

@hebasto
Copy link
Member Author

hebasto commented May 8, 2023

@pinheadmz

Actually I still get the untracked files warning after merge -- am i doing something wrong?

Try to clean your local repo first. For example, git clean -xdff.

@hebasto hebasto deleted the 230505-cleanup branch May 8, 2023 15:20
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 8, 2023
@bitcoin bitcoin locked and limited conversation to collaborators May 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants