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

depends: qt: avoid system harfbuzz and bz2 #14385

Merged
merged 1 commit into from Oct 8, 2018

Conversation

@theuni
Copy link
Member

@theuni theuni commented Oct 3, 2018

Should fix #14367.

Gitian builds end up with this config implicitly due to missing harfbuzz system lib, this change explicitly disables the use of the lib completely, even if present.

We may eventually want to break out harfbuzz and build it in depends, but for now just ensure that runtime dependencies don't depend on whether or not harfbuzz was present on the builder.

@real-or-random Can you confirm that this fixes your issue?

We may eventually want to break out harfbuzz and build it in depends, but
for now just ensure that runtime dependencies don't depend on whether or not
harfbuzz was present on the builder.
@DrahtBot
Copy link
Contributor

@DrahtBot DrahtBot commented Oct 4, 2018

Coverage Change (pull 14385) Reference (master)
Lines +0.0419 % 86.9985 %
Functions -0.0154 % 84.1902 %
Branches +0.0398 % 51.5024 %

@real-or-random
Copy link
Contributor

@real-or-random real-or-random commented Oct 4, 2018

Thanks, this fixes the build for me. 👍

(But I can't judge how bad this affects users for which harfbuzz works currently.)

@DrahtBot
Copy link
Contributor

@DrahtBot DrahtBot commented Oct 4, 2018

Gitian builds for commit 1f59c6f (master):

Gitian builds for commit afaea21ac6980fb09750e76cb327430a84899355 (master and this pull):

@bitcoin bitcoin deleted a comment from STALININST Oct 4, 2018
@ken2812221
Copy link
Contributor

@ken2812221 ken2812221 commented Oct 6, 2018

utACK f149e31

@fanquake
Copy link
Member

@fanquake fanquake commented Oct 8, 2018

utACK f149e31

This upstream ticket also partially explains what happened (I haven't tracked down the actual commit). Basically, from 5.8.0, Qt overhauled its build system, and now defaults to -system-harfbuzz.

Also discussed with @theuni, and in any case we definitely don't want to be pulling in system harfbuzz, so making this explicit is better.

@MarcoFalke MarcoFalke merged commit f149e31 into bitcoin:master Oct 8, 2018
2 checks passed
MarcoFalke added a commit that referenced this issue Oct 8, 2018
f149e31 depends: qt: avoid system harfbuzz and bz2 (Cory Fields)

Pull request description:

  Should fix #14367.

  Gitian builds end up with this config implicitly due to missing harfbuzz system lib, this change explicitly disables the use of the lib completely, even if present.

  We may eventually want to break out harfbuzz and build it in depends, but for now just ensure that runtime dependencies don't depend on whether or not harfbuzz was present on the builder.

  @real-or-random Can you confirm that this fixes your issue?

Tree-SHA512: 1ad3dd572c1f2004ec80ebaf3b6831327b0267fb3dc1895e77bd04785cd6e32bafc931932195462bbae807289bc565f0ea740f4050d26f0233ddcd62bfdb0685
@bitcoin bitcoin deleted a comment from STALININST Oct 13, 2018
UdjinM6 added a commit to dashpay/dash that referenced this issue Mar 1, 2019
We may eventually want to break out harfbuzz and build it in depends, but
for now just ensure that runtime dependencies don't depend on whether or not
harfbuzz was present on the builder.
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this issue Mar 10, 2019
We may eventually want to break out harfbuzz and build it in depends, but
for now just ensure that runtime dependencies don't depend on whether or not
harfbuzz was present on the builder.
TriKriSta referenced this issue in ivansib/sibcoin Jul 4, 2019
We may eventually want to break out harfbuzz and build it in depends, but
for now just ensure that runtime dependencies don't depend on whether or not
harfbuzz was present on the builder.
koad pushed a commit to Canada-eCoin/eCoinCore that referenced this issue Oct 9, 2020
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

6 participants