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

Upstream PRs 1056, 1104, 1105, 1084, 1114, 1115, 1116, 1120, 1122, 1121, 1128, 1131, 1144, 1150, 1146 #238

Merged

Commits on Jul 1, 2022

  1. build: Fix #include "..." paths to get rid of further -I arguments

    This simplifies building without a build system.
    
    This is in line with #925; the paths fixed here were either forgotten
    there or only introduced later. This commit also makes the Makefile
    stricter so that further "wrong" #include paths will lead to build
    errors even in autotools builds.
    
    This belongs to #929.
    
    Co-authored-by: Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>
    real-or-random and hebasto committed Jul 1, 2022
    Configuration menu
    Copy the full SHA
    40a3473 View commit details
    Browse the repository at this point in the history
  2. Merge bitcoin-core/secp256k1#1116: build: Fix #include "..." paths to…

    … get rid of further -I arguments
    
    40a3473 build: Fix #include "..." paths to get rid of further -I arguments (Tim Ruffing)
    
    Pull request description:
    
      This simplifies building without a build system.
    
      This is in line with #925; the paths fixed here were either forgotten
      there or only introduced later. This commit also makes the Makefile
      stricter so that further "wrong" #include paths will lead to build
      errors even in autotools builds.
    
      This belongs to #929.
    
    ACKs for top commit:
      hebasto:
        ACK 40a3473
    
    Tree-SHA512: 6f4d825ea3cf86b13f294e2ec19fafc29660fa99450e6b579157d7a6e9bdb3404d761edf89c1135fa89b984d6431a527beeb97031dc90f2fae9761528f4d06d1
    real-or-random committed Jul 1, 2022
    Configuration menu
    Copy the full SHA
    af65d30 View commit details
    Browse the repository at this point in the history

Commits on Jul 5, 2022

  1. ecmult_gen: Skip RNG when creating blinding if no seed is available

    Running the RNG is pointless if no seed is available because the key
    will be fixed. The computation just wastes time.
    
    Previously, users could avoid this computation at least by asking for
    a context without signing capabilities. But since 3b0c218 we always
    build an ecmult_gen context, ignoring the context flags. Moreover,
    users could never avoid this pointless computation when asking for
    the creation of a signing context.
    real-or-random committed Jul 5, 2022
    Configuration menu
    Copy the full SHA
    4cc0b1b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    7a86955 View commit details
    Browse the repository at this point in the history
  3. ecmult_gen: Improve comments about projective blinding

    Whenever I read this code, I first think that rescaling ctx->initial is
    a dead store because we overwrite it later with gb. But that's wrong.
    The rescaling blinds the computation of gb and affects its result.
    real-or-random committed Jul 5, 2022
    Configuration menu
    Copy the full SHA
    55f8bc9 View commit details
    Browse the repository at this point in the history

Commits on Jul 6, 2022

  1. config: Set preprocessor defaults for ECMULT_* config values

    This simplifies manual builds and solves one item in #929.
    real-or-random committed Jul 6, 2022
    Configuration menu
    Copy the full SHA
    d0cf55e View commit details
    Browse the repository at this point in the history

Commits on Jul 7, 2022

  1. Merge bitcoin-core/secp256k1#1120: ecmult_gen: Skip RNG when creating…

    … blinding if no seed is available
    
    55f8bc9 ecmult_gen: Improve comments about projective blinding (Tim Ruffing)
    7a86955 ecmult_gen: Simplify code (no observable change) (Tim Ruffing)
    4cc0b1b ecmult_gen: Skip RNG when creating blinding if no seed is available (Tim Ruffing)
    
    Pull request description:
    
      Running the RNG is pointless if no seed is available because the key
      will be fixed. The computation just wastes time.
    
      Previously, users could avoid this computation at least by asking for
      a context without signing capabilities. But since 3b0c218 we always
      build an ecmult_gen context, ignoring the context flags. Moreover,
      users could never avoid this pointless computation when asking for
      the creation of a signing context.
    
      This fixes one item in #1065.
    
    ACKs for top commit:
      sipa:
        ACK 55f8bc9
      apoelstra:
        ACK 55f8bc9
    
    Tree-SHA512: 5ccba56041f94fa8f40a8a56ce505369ff2e0ed20cd7f0bfc3fdfffa5fa7bf826a93602b9b2455a352865a9548ab4928e858c19bb5af7ec221594a3bf25c4f3d
    real-or-random committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    63a3565 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    da6514a View commit details
    Browse the repository at this point in the history
  3. config: Remove basic-config.h

    It's unused and thus potentially confusing.
    real-or-random committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    c27ae45 View commit details
    Browse the repository at this point in the history

Commits on Jul 8, 2022

  1. Configuration menu
    Copy the full SHA
    17065f4 View commit details
    Browse the repository at this point in the history

Commits on Jul 11, 2022

  1. Merge bitcoin-core/secp256k1#1122: tests: Randomize the context with …

    …probability 15/16 instead of 1/4
    
    17065f4 tests: Randomize the context with probability 15/16 instead of 1/4 (Tim Ruffing)
    
    Pull request description:
    
    ACKs for top commit:
      sipa:
        ACK 17065f4
      jonasnick:
        ACK 17065f4
    
    Tree-SHA512: 3b7005770007b922a294be610f23da60b0dde74dfd7585d64a2cb04eaa6ec879de8d21a0ade31c1857019a8dd97260fa3aa167ae16fc55027ef280a3e3feaa6d
    jonasnick committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    6a873cc View commit details
    Browse the repository at this point in the history
  2. Merge bitcoin-core/secp256k1#1121: config: Set preprocessor defaults …

    …for ECMULT_* config values
    
    c27ae45 config: Remove basic-config.h (Tim Ruffing)
    da6514a config: Introduce DEBUG_CONFIG macro for debug output of config (Tim Ruffing)
    d0cf55e config: Set preprocessor defaults for ECMULT_* config values (Tim Ruffing)
    
    Pull request description:
    
    ACKs for top commit:
      sipa:
        ACK c27ae45
      hebasto:
        ACK c27ae45, I have reviewed the code and it looks correct.
      jonasnick:
        ACK c27ae45
    
    Tree-SHA512: 56b0f384bd9f42cf7c903bec08f4807db1415ddf9a06676dfe1e638e4d02431c522ef0422585e85429074e0dbb51da4f400cf53e8f883d6e07122731c57be1e3
    jonasnick committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    3efeb9d View commit details
    Browse the repository at this point in the history

Commits on Jul 21, 2022

  1. configure: Remove pkgconfig macros again (reintroduced by mismerge)

    We had removed `PKG_PROG_PKG_CONFIG` in 21b2eba
    (#1090). But then then the not rebased (!) merge of 2be6ba0
    (#1084) brought that macro back at another location, without git
    complaining about a conflict.
    
    Fixes #1127.
    real-or-random committed Jul 21, 2022
    Configuration menu
    Copy the full SHA
    cabe085 View commit details
    Browse the repository at this point in the history
  2. Merge bitcoin-core/secp256k1#1128: configure: Remove pkgconfig macros…

    … again (reintroduced by mismerge)
    
    cabe085 configure: Remove pkgconfig macros again (reintroduced by mismerge) (Tim Ruffing)
    
    Pull request description:
    
      We had removed `PKG_PROG_PKG_CONFIG` in 21b2eba
      (#1090). But then then the not rebased (!) merge of 2be6ba0
      (#1084) brought that macro back at another location, without git
      complaining about a conflict.
    
      Fixes #1127.
    
    ACKs for top commit:
      fanquake:
        ACK cabe085
      hebasto:
        ACK cabe085
      jonasnick:
        ACK cabe085
    
    Tree-SHA512: ba497503db3a11e631b15c4fe875e62d892971c2c708d90b2f6be684e85d164043ea97c13af0452831eef41f3cf8230cd8a9eafa332dc5b5ae18e118b87c3828
    real-or-random committed Jul 21, 2022
    Configuration menu
    Copy the full SHA
    9f8a13d View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2022

  1. Configuration menu
    Copy the full SHA
    ef48f08 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    78f5296 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    88b0089 View commit details
    Browse the repository at this point in the history
  4. Merge bitcoin-core/secp256k1#1131: readme: Misc improvements

    88b0089 readme: Fix line break (Tim Ruffing)
    78f5296 readme: Sell "no runtime dependencies" (Tim Ruffing)
    ef48f08 readme: Add IRC channel (Tim Ruffing)
    
    Pull request description:
    
    ACKs for top commit:
      apoelstra:
        utACK 88b0089
      sipa:
        ACK 88b0089
    
    Tree-SHA512: 174f1596406f98a19059a18cd4fb993102e5ffb8ec29fcc6d03e27f135fcb526b37204b64055b5e4f0a273daab05d395cf335f26241cf3a29a060041c9ef109b
    real-or-random committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    694ce8f View commit details
    Browse the repository at this point in the history

Commits on Oct 28, 2022

  1. Revert "Add test logs to gitignore"

    This reverts commit bceefd6.
    hebasto committed Oct 28, 2022
    Configuration menu
    Copy the full SHA
    798727a View commit details
    Browse the repository at this point in the history
  2. Cleanup .gitignore file

    The removed line was introduced for `obj/.gitignore` file. Since the
    `obj` directory has been removed, it is not longer required.
    hebasto committed Oct 28, 2022
    Configuration menu
    Copy the full SHA
    f5039cb View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2022

  1. Merge bitcoin-core/secp256k1#1144: Cleanup .gitignore file

    f5039cb Cleanup `.gitignore` file (Hennadii Stepanov)
    798727a Revert "Add test logs to gitignore" (Hennadii Stepanov)
    
    Pull request description:
    
    ACKs for top commit:
      jonasnick:
        ACK f5039cb
      real-or-random:
        ACK f5039cb
    
    Tree-SHA512: 3586329e77958a9bfa06dd84e5b121cd456e93332670d5afc1a6691e165cdfa5a6fd6a61f82be12ec33f2a58b26a13adfedeb177ae1056202e53a530949fc549
    jonasnick committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    a43e982 View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2022

  1. ci: mostly prevent "-v/--version: not found" irrelevant error

    $CC, $WRAPPER_CMD and valgrind are not necessarily defined
    jonasnick committed Nov 2, 2022
    Configuration menu
    Copy the full SHA
    49ae843 View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2022

  1. Configuration menu
    Copy the full SHA
    5c9f1a5 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2022

  1. Merge bitcoin-core/secp256k1#1150: ci: always cat test_env.log

    5c9f1a5 ci: always cat all logs_snippets (Jonas Nick)
    
    Pull request description:
    
    ACKs for top commit:
      real-or-random:
        ACK 5c9f1a5
    
    Tree-SHA512: fc715c5fc4006d80a4b0c2faa8ef81ed23c4479614945ffa7c96665a3acc38fe956dd6d148bcf97043232ceee055b724ea9490e4ac4142a210e4488fed8dd299
    jonasnick committed Nov 8, 2022
    Configuration menu
    Copy the full SHA
    e13fae4 View commit details
    Browse the repository at this point in the history
  2. Merge bitcoin-core/secp256k1#1146: ci: prevent "-v/--version: not fou…

    …nd" irrelevant error
    
    49ae843 ci: mostly prevent "-v/--version: not found" irrelevant error (Jonas Nick)
    
    Pull request description:
    
    ACKs for top commit:
      real-or-random:
        ACK 49ae843
    
    Tree-SHA512: 2e048b037826dff372e26103f198e0d490494e7909d17d8632b51f6d9e0629b51bcd0b55b65b2c21d63d522394ccfed481ce126cea165c087df670556bc8ccf6
    jonasnick committed Nov 8, 2022
    Configuration menu
    Copy the full SHA
    c2ee917 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2023

  1. Configuration menu
    Copy the full SHA
    64717a7 View commit details
    Browse the repository at this point in the history