Skip to content

Conversation

brakmic
Copy link
Contributor

@brakmic brakmic commented Mar 25, 2020

This PR removes the redundant LIBBITCOIN_SERVER linking from bench's Makefile.
This PR is similar to #17910

Originally, this PR was part of #18377, which later got replaced by a better one #18397 written by hebasto.

@laanwj
Copy link
Member

laanwj commented Mar 25, 2020

Concept ACK

@Empact
Copy link
Contributor

Empact commented Mar 25, 2020

Code Review ACK 1f97b69

@sipa
Copy link
Member

sipa commented Mar 25, 2020

ACK unless someone sees a reason why this would break in certain untested environments.

@DrahtBot
Copy link
Contributor

Gitian builds

File commit 3f5107d
(master)
commit 5876e0d
(master and this pull)
bitcoin-0.19.99-aarch64-linux-gnu-debug.tar.gz c54ba24037a5f515... baa74f743bcc60e9...
bitcoin-0.19.99-aarch64-linux-gnu.tar.gz af9ed3afb2470996... 967f274a0bbd0700...
bitcoin-0.19.99-arm-linux-gnueabihf-debug.tar.gz 5f75367316182cda... 2c0396fa13b331d4...
bitcoin-0.19.99-arm-linux-gnueabihf.tar.gz 09130a219271d5e9... 0c5a53e09438e89c...
bitcoin-0.19.99-osx-unsigned.dmg d7215c3115ae6238... 98a4fce361a3d2cd...
bitcoin-0.19.99-osx64.tar.gz b7bcf04821dfdefe... ae59e77a24c3cd3a...
bitcoin-0.19.99-riscv64-linux-gnu-debug.tar.gz 0fff31b63f9780bc... 9558bc495130e0c2...
bitcoin-0.19.99-riscv64-linux-gnu.tar.gz 9b711a748c74a876... ea5e74529d7f5667...
bitcoin-0.19.99-win64-debug.zip 6db3ea24a8fec68b... 537a0702eb11e618...
bitcoin-0.19.99-win64-setup-unsigned.exe ba02debc356a9b25... c33ed0d86a7ed0c0...
bitcoin-0.19.99-win64.zip 13ce9ce2c30a9911... 41f19446c697268b...
bitcoin-0.19.99-x86_64-linux-gnu-debug.tar.gz 05edff0962f27dbd... e883d9b99b354d8b...
bitcoin-0.19.99-x86_64-linux-gnu.tar.gz ead201970ef63f4c... 98aa771bf5399261...
bitcoin-0.19.99.tar.gz cd3c1ff29350248f... 5f1fae0019b0886e...
bitcoin-core-linux-0.20-res.yml 888749de8538c24f... 795ca6ba8a59d0e7...
bitcoin-core-osx-0.20-res.yml a2f78c976c2a71c2... 7fcbe16210a1f028...
bitcoin-core-win-0.20-res.yml d36622eddbb89bd0... c95dab3d2e031138...
linux-build.log 5b5f98c9fb04c753... bcb4d0650c54e711...
osx-build.log 61646ea5b19a8042... e43c671e45e5dc47...
win-build.log 577098ae11ed401b... e237afd64cdbdc2a...
bitcoin-core-linux-0.20-res.yml.diff bc0d8a6ad60374ed...
bitcoin-core-osx-0.20-res.yml.diff 9ce5ce668425dff2...
bitcoin-core-win-0.20-res.yml.diff 6164fa6f0798d9a7...
linux-build.log.diff 9fa9e0347f396ac6...
osx-build.log.diff 11e05e8404765fb7...
win-build.log.diff af4f016f01291092...

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK 1f97b69

Copy link
Contributor

@theStack theStack left a comment

Choose a reason for hiding this comment

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

ACK 1f97b69

@fanquake fanquake merged commit e99ca78 into bitcoin:master Mar 26, 2020
MarkLTZ added a commit to litecoinz-core/litecoinz that referenced this pull request Mar 27, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 28, 2020
…h-Makefile

1f97b69 build: remove double LIBBITCOIN_SERVER from bench-Makefile (Harris)

Pull request description:

  This PR removes the redundant **LIBBITCOIN_SERVER** linking from bench's Makefile.
  This PR is similar to bitcoin#17910

  Originally, this PR was part of bitcoin#18377, which later got replaced by a better one bitcoin#18397 written by **hebasto**.

ACKs for top commit:
  Empact:
    Code Review ACK bitcoin@1f97b69
  theStack:
    ACK bitcoin@1f97b69
  hebasto:
    ACK 1f97b69

Tree-SHA512: e43035262361d4458a7dcfc920445540f19301387814cde1be0539c936fc20da0dcbe49e5ea25385e6d36d9639515b7a4171228223da568d93427e9c32810945
MarkLTZ added a commit to litecoinz-core/litecoinz that referenced this pull request Apr 6, 2020
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Nov 1, 2021
…h-Makefile

1f97b69 build: remove double LIBBITCOIN_SERVER from bench-Makefile (Harris)

Pull request description:

  This PR removes the redundant **LIBBITCOIN_SERVER** linking from bench's Makefile.
  This PR is similar to bitcoin#17910

  Originally, this PR was part of bitcoin#18377, which later got replaced by a better one bitcoin#18397 written by **hebasto**.

ACKs for top commit:
  Empact:
    Code Review ACK bitcoin@1f97b69
  theStack:
    ACK bitcoin@1f97b69
  hebasto:
    ACK 1f97b69

Tree-SHA512: e43035262361d4458a7dcfc920445540f19301387814cde1be0539c936fc20da0dcbe49e5ea25385e6d36d9639515b7a4171228223da568d93427e9c32810945
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Dec 22, 2021
…h-Makefile

1f97b69 build: remove double LIBBITCOIN_SERVER from bench-Makefile (Harris)

Pull request description:

  This PR removes the redundant **LIBBITCOIN_SERVER** linking from bench's Makefile.
  This PR is similar to bitcoin#17910

  Originally, this PR was part of bitcoin#18377, which later got replaced by a better one bitcoin#18397 written by **hebasto**.

ACKs for top commit:
  Empact:
    Code Review ACK bitcoin@1f97b69
  theStack:
    ACK bitcoin@1f97b69
  hebasto:
    ACK 1f97b69

Tree-SHA512: e43035262361d4458a7dcfc920445540f19301387814cde1be0539c936fc20da0dcbe49e5ea25385e6d36d9639515b7a4171228223da568d93427e9c32810945
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Dec 22, 2021
…h-Makefile

1f97b69 build: remove double LIBBITCOIN_SERVER from bench-Makefile (Harris)

Pull request description:

  This PR removes the redundant **LIBBITCOIN_SERVER** linking from bench's Makefile.
  This PR is similar to bitcoin#17910

  Originally, this PR was part of bitcoin#18377, which later got replaced by a better one bitcoin#18397 written by **hebasto**.

ACKs for top commit:
  Empact:
    Code Review ACK bitcoin@1f97b69
  theStack:
    ACK bitcoin@1f97b69
  hebasto:
    ACK 1f97b69

Tree-SHA512: e43035262361d4458a7dcfc920445540f19301387814cde1be0539c936fc20da0dcbe49e5ea25385e6d36d9639515b7a4171228223da568d93427e9c32810945
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants