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

build: remove double LIBBITCOIN_SERVER linking #17910

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

fanquake
Copy link
Member

Seems that this is no longer required. Have tested building on macOS and Debian.

@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #16367 (Multiprocess build support by ryanofsky)
  • #10102 ([experimental] Multiprocess bitcoin by ryanofsky)

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.

@hebasto
Copy link
Member

hebasto commented Jan 12, 2020

Seems that this is no longer required.

Is it a result of #15638 ?

@DrahtBot
Copy link
Contributor

Gitian builds

File commit e258ce7
(master)
commit 4f85a32
(master and this pull)
bitcoin-0.19.99-aarch64-linux-gnu-debug.tar.gz 048c99029d1f00d0... d3675fd0fefb5f08...
bitcoin-0.19.99-aarch64-linux-gnu.tar.gz f5f50540f6c70397... 04312cf42260af6a...
bitcoin-0.19.99-arm-linux-gnueabihf-debug.tar.gz 888ca1ceac12de4a... 2a1aef520723001a...
bitcoin-0.19.99-arm-linux-gnueabihf.tar.gz 484496fe668e1af6... 817e98eb6717cdd0...
bitcoin-0.19.99-i686-pc-linux-gnu-debug.tar.gz a6b56561f7134430... f5d257ea11f12875...
bitcoin-0.19.99-i686-pc-linux-gnu.tar.gz c72c6e465f9a8511... c348a319a6693cf2...
bitcoin-0.19.99-osx-unsigned.dmg 3e4ad919f64575b4... 582c46f93e2ef245...
bitcoin-0.19.99-osx64.tar.gz a286a84b4f1d6d7e... a14649dbab7c3e6f...
bitcoin-0.19.99-riscv64-linux-gnu-debug.tar.gz ae39ebf0b8f7a5b2... 3a527ebcc47a9cda...
bitcoin-0.19.99-riscv64-linux-gnu.tar.gz 94a676af2324957c... 98a01fa855b8d937...
bitcoin-0.19.99-win64-debug.zip 28d487d8150eb217... d7ed61bb7e94412d...
bitcoin-0.19.99-win64-setup-unsigned.exe c5536af10a6c637f... 5863989526e3cf64...
bitcoin-0.19.99-win64.zip 3ff5d481fec17519... 18b6617834ec8092...
bitcoin-0.19.99-x86_64-linux-gnu-debug.tar.gz 218f61bb1a4de3e5... c2fb4cc2b90ac0ed...
bitcoin-0.19.99-x86_64-linux-gnu.tar.gz 07e0bd193b461042... 318f7cbd08765182...
bitcoin-0.19.99.tar.gz 4d46a6481b6711f5... 266d7e113bf50710...
bitcoin-core-linux-0.20-res.yml a1008e30a305dacd... d20abb285fc4716d...
bitcoin-core-osx-0.20-res.yml 8e6e044e8ffea3af... 855cc60e8a5b57a5...
bitcoin-core-win-0.20-res.yml 7f7e7a30ff49851d... 168b18f1b5d51590...
linux-build.log 46a2ee98f7351bfc... b99083110a6db501...
osx-build.log 5f8c87a00c40573d... 33bebe4e3ce62976...
win-build.log 546e31c3bb6b1ab8... 76e78a40f75aee2f...
bitcoin-core-linux-0.20-res.yml.diff d9d7bba3b91863e4...
bitcoin-core-osx-0.20-res.yml.diff 4505db509098dafe...
bitcoin-core-win-0.20-res.yml.diff 62bbf2a732febafb...
linux-build.log.diff f14c857980fb55a9...
osx-build.log.diff fd0a2d602bd93793...
win-build.log.diff df43d83f9c139bea...

@promag
Copy link
Member

promag commented Jan 12, 2020

ACK 831e122.

@practicalswift
Copy link
Contributor

ACK 831e122

@laanwj
Copy link
Member

laanwj commented Jan 13, 2020

good catch, I'm happy that this is no longer needed!
ACK 831e122

laanwj added a commit that referenced this pull request Jan 13, 2020
831e122 build: remove double LIBBITCOIN_SERVER linking (fanquake)

Pull request description:

  Seems that this is no longer required. Have tested building on macOS and Debian.

ACKs for top commit:
  promag:
    ACK 831e122.
  practicalswift:
    ACK 831e122
  laanwj:
    ACK 831e122

Tree-SHA512: d226d9fa0292189fae7e2af14781a511c3633f1352324f19ae642e941d06c34e2abf8b1df97d2330d76dba6024a93d8d341e02cc4882d7066f97e82585631fe1
@laanwj laanwj merged commit 831e122 into bitcoin:master Jan 13, 2020
@fanquake fanquake deleted the remove_double_server branch January 13, 2020 10:48
@fanquake fanquake removed the request for review from ryanofsky January 13, 2020 10:48
MarkLTZ added a commit to litecoinz-core/litecoinz that referenced this pull request Jan 21, 2020
fanquake added a commit that referenced this pull request Mar 26, 2020
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 #17910

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

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

Tree-SHA512: e43035262361d4458a7dcfc920445540f19301387814cde1be0539c936fc20da0dcbe49e5ea25385e6d36d9639515b7a4171228223da568d93427e9c32810945
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
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.

None yet

6 participants