Skip to content

Conversation

fanquake
Copy link
Member

@fanquake fanquake commented Mar 3, 2021

Brew has updated such that qt now refers to Qt 6.0.1. If builders
install this, configure will not pick up qt. For now, install
qt@5 (5.15.2), until required build system and likely source changes
are made.

Brew has updated such that qt now refers to Qt 6.0.1. If builders
install this, configure will not work pick up qt. For now, install
qt@5 (5.15.2), until required build system and likely source changes
are made.
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 bec7f2c, tested on Tested on macOS Big Sur 11.2.2 (20D80).

Running brew update I can see:

% brew update
...
==> Updated Formulae
qt ✔️                       
...
==> Renamed Formulae
qt5 -> qt@5
...

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 3, 2021

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

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.

@maflcko
Copy link
Member

maflcko commented Mar 3, 2021

Seems fine to temporarily pin the version if compilation with 6.0 fails

@maflcko maflcko merged commit 9212e67 into bitcoin:master Mar 3, 2021
@fanquake fanquake deleted the explicit_qt5 branch March 3, 2021 09:38
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 3, 2021
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Mar 11, 2021
Summary:
```
Brew has updated such that qt now refers to Qt 6.0.1. If builders
install this, configure will not work pick up qt. For now, install
qt@5 (5.15.2), until required build system and likely source changes
are made.
```

Backport of [[ bitcoin/bitcoin#21346 | core#21346 ]].

Test Plan:
On OSX:
  brew update
  brew info qt # returns a qt 6.x.x version
  brew info qt@5 # returns a qt 5.x.x version

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

Differential Revision: https://reviews.bitcoinabc.org/D9317
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 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.

4 participants