msvc: Cleanup after upgrading libsecp256k1 up to 0.3.0 #27580

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:230505-cleanup changing 3 files +1 −16
  1. hebasto commented at 10:09 AM on May 5, 2023: member

    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.

  2. msvc: Cleanup after upgrading libsecp256k1 up to 0.3.0 d9b54c46cc
  3. DrahtBot commented at 10:09 AM on May 5, 2023: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--021abf342d371248e50ceaed478a90ca-->

    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.

    <!--174a7506f384e20aa4161008e828411d-->

    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.

  4. fanquake approved
  5. fanquake commented at 11:05 AM on May 6, 2023: member

    ACK d9b54c46ccb28af20eb03e1409d1a34dc2adccdb

  6. fanquake commented at 11:13 AM on May 6, 2023: member

    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.

  7. fanquake merged this on May 8, 2023
  8. fanquake closed this on May 8, 2023

  9. pinheadmz commented at 2:13 PM on May 8, 2023: 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)
    
  10. pinheadmz commented at 2:21 PM on May 8, 2023: 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?

  11. hebasto commented at 3:03 PM on May 8, 2023: member

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

  12. hebasto deleted the branch on May 8, 2023
  13. sidhujag referenced this in commit 13b0b69e94 on May 8, 2023
  14. bitcoin locked this on May 7, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-19 12:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me