Skip to content

Conversation

fanquake
Copy link
Member

@fanquake fanquake commented Dec 9, 2019

xcb_proto's configure doesn't understand --disable-shared or
--with-pic. All the package does it put a stack of XML files into
a directory to be used by libxcb.

Probably enough to close #16354.

xcb_proto's configure doesn't understand --disable-shared or
--with-pic. All the package does it put a stack of xml files into
a directory to be used by libxcb.
@laanwj
Copy link
Member

laanwj commented Dec 9, 2019

xcb-proto provides the XML-XCB protocol descriptions that libxcb uses to
generate the majority of its code and API. We provide them separately
from libxcb to allow reuse by other projects, such as additional
language bindings, protocol dissectors, or documentation generators.

Concept ACK

@dongcarl
Copy link
Contributor

dongcarl commented Dec 9, 2019

ACK e97f5c1

@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 9, 2019

Gitian builds

File commit bb03765
(master)
commit 7b20ec5
(master and this pull)
bitcoin-0.19.99-aarch64-linux-gnu-debug.tar.gz 8240455bc3e5bfac... b43ab153c3615635...
bitcoin-0.19.99-aarch64-linux-gnu.tar.gz 2143342645b444b3... 068cfdf9f817c97e...
bitcoin-0.19.99-arm-linux-gnueabihf-debug.tar.gz 1752f8a580b62f62... de589df4bca5a6f6...
bitcoin-0.19.99-arm-linux-gnueabihf.tar.gz ce66fd43a8668fb2... bbdcfd89bc32f1d2...
bitcoin-0.19.99-i686-pc-linux-gnu-debug.tar.gz c9dc6ba2f2d43aa2... 4f1ced9a54670fbe...
bitcoin-0.19.99-i686-pc-linux-gnu.tar.gz 4e432b389d85c1fd... 5602d391e49918dc...
bitcoin-0.19.99-osx-unsigned.dmg a94deff32760864a... 17d6a1757b67b6bf...
bitcoin-0.19.99-osx64.tar.gz 7fb753459b33cd2b... e89f87214bc9bfa0...
bitcoin-0.19.99-riscv64-linux-gnu-debug.tar.gz 5c37ba2e2f332f2d... 4e172867ece970b0...
bitcoin-0.19.99-riscv64-linux-gnu.tar.gz a71538890d2163fb... 65092b9b48fd2b52...
bitcoin-0.19.99-win64-debug.zip 680114171021257d... a5ba1542f7e9c0b5...
bitcoin-0.19.99-win64-setup-unsigned.exe c87334eea1f093a2... 11af83932e6e9dee...
bitcoin-0.19.99-win64.zip 3c71341450aed9f5... d3ebad7fa1c0944a...
bitcoin-0.19.99-x86_64-linux-gnu-debug.tar.gz 1bf554e99cbd3c8c... 883cb02ff481cef4...
bitcoin-0.19.99-x86_64-linux-gnu.tar.gz 8c6b980666e2a61f... 3e2e0551907f122f...
bitcoin-0.19.99.tar.gz 246e7c0f86bcb4a5... c446beb4200291d4...
bitcoin-core-linux-0.20-res.yml ed8185eb1ef571ff... 700d3570c077e7dd...
bitcoin-core-osx-0.20-res.yml b5a60c2c988e0e02... 13807d6e5a70c4f0...
bitcoin-core-win-0.20-res.yml aa535c79db558dda... 5ec9194c0ebaa99b...
linux-build.log 02643a98d0e1c2cf... 3cc396441dfb88f9...
osx-build.log 2eeef83d97f86634... c94a27d5e54b4de8...
win-build.log 096f1d712e55e6b6... d390d164a1cfcb2f...
bitcoin-core-linux-0.20-res.yml.diff 8e232935533f9abc...
bitcoin-core-osx-0.20-res.yml.diff 321ccdf1876b18ba...
bitcoin-core-win-0.20-res.yml.diff 804cff5ec235ca1f...
linux-build.log.diff d7f4bf292dec1115...
osx-build.log.diff 2b49ee0b33d3307c...
win-build.log.diff af5e4dc7b7268972...

laanwj added a commit that referenced this pull request Dec 11, 2019
e97f5c1 depends: don't configure xcb_proto (fanquake)

Pull request description:

  xcb_proto's configure doesn't understand `--disable-shared` or
  `--with-pic`. All the package does it put a stack of XML files into
  a directory to be used by libxcb.

  Probably enough to close #16354.

ACKs for top commit:
  dongcarl:
    ACK e97f5c1

Tree-SHA512: 1a49fd7c8269405bbf312be33c1aeaac5f25ef8666829b01dc3c58f3a2a9281c23c42614a7f1cfc3ee260be4ea3e71285869b1cb9c2035dceda336296d9d9dea
@laanwj laanwj merged commit e97f5c1 into bitcoin:master Dec 11, 2019
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 12, 2019
e97f5c1 depends: don't configure xcb_proto (fanquake)

Pull request description:

  xcb_proto's configure doesn't understand `--disable-shared` or
  `--with-pic`. All the package does it put a stack of XML files into
  a directory to be used by libxcb.

  Probably enough to close bitcoin#16354.

ACKs for top commit:
  dongcarl:
    ACK e97f5c1

Tree-SHA512: 1a49fd7c8269405bbf312be33c1aeaac5f25ef8666829b01dc3c58f3a2a9281c23c42614a7f1cfc3ee260be4ea3e71285869b1cb9c2035dceda336296d9d9dea
@fanquake fanquake deleted the xcbproto_no_disable_no_pic branch March 3, 2020 00:29
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Apr 3, 2020
Summary:
```
xcb_proto's configure doesn't understand --disable-shared or
--with-pic. All the package does it put a stack of xml files into
a directory to be used by libxcb.
```

Backport of core [[bitcoin/bitcoin#17698 | PR17698]].

Test Plan: Run the Linux Gitian builds.

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D5667
sidhujag pushed a commit to syscoin-core/syscoin that referenced this pull request Nov 10, 2020
e97f5c1 depends: don't configure xcb_proto (fanquake)

Pull request description:

  xcb_proto's configure doesn't understand `--disable-shared` or
  `--with-pic`. All the package does it put a stack of XML files into
  a directory to be used by libxcb.

  Probably enough to close bitcoin#16354.

ACKs for top commit:
  dongcarl:
    ACK e97f5c1

Tree-SHA512: 1a49fd7c8269405bbf312be33c1aeaac5f25ef8666829b01dc3c58f3a2a9281c23c42614a7f1cfc3ee260be4ea3e71285869b1cb9c2035dceda336296d9d9dea
@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.

build: Review of depends tree configure flags
4 participants