doc: Update minimum required Qt #14909

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:doc-qt-minimum-52 changing 1 files +4 −4

fanquake commented at 6:58 AM on December 10, 2018: member

After #14725 the minimum required version of Qt is 5.2.
doc: Update minimum required qt b7bee6af76
fanquake added the label Docs on Dec 10, 2018

in doc/dependencies.md:16 in b7bee6af76

  14 |  | Expat | [2.2.5](https://libexpat.github.io/) |  | No | Yes |  |
  15 |  | fontconfig | [2.12.1](https://www.freedesktop.org/software/fontconfig/release/) |  | No | Yes |  |
  16 | -| FreeType | [2.7.1](http://download.savannah.gnu.org/releases/freetype) |  | No |  |  |
  17 | +| FreeType | [2.7.1](https://download.savannah.gnu.org/releases/freetype) |  | No |  |  |
  18 |  | GCC |  | [4.8+](https://gcc.gnu.org/) (C++11 support) |  |  |  |
  19 |  | HarfBuzz-NG |  |  |  |  |  |

hebasto commented at 7:50 AM on December 10, 2018:

 | HarfBuzz-NG | | | | | [Yes](https://github.com/bitcoin/bitcoin/blob/master/depends/packages/qt.mk#L67) |

in doc/dependencies.md:14 in b7bee6af76

  12 |  | Clang |  | [3.3+](https://llvm.org/releases/download.html) (C++11 support) |  |  |  |
  13 |  | D-Bus | [1.10.18](https://cgit.freedesktop.org/dbus/dbus/tree/NEWS?h=dbus-1.10) |  | No | Yes |  |
  14 |  | Expat | [2.2.5](https://libexpat.github.io/) |  | No | Yes |  |
  15 |  | fontconfig | [2.12.1](https://www.freedesktop.org/software/fontconfig/release/) |  | No | Yes |  |
  16 | -| FreeType | [2.7.1](http://download.savannah.gnu.org/releases/freetype) |  | No |  |  |
  17 | +| FreeType | [2.7.1](https://download.savannah.gnu.org/releases/freetype) |  | No |  |  |

hebasto commented at 8:03 AM on December 10, 2018:

| FreeType | [2.7.1](https://download.savannah.gnu.org/releases/freetype) |  | No |  | [No](https://github.com/bitcoin/bitcoin/blob/master/depends/packages/qt.mk#L33) |

And a similar change for zlib?

hebasto changes_requested
hebasto commented at 8:06 AM on December 10, 2018: member

Concept ACK.
DrahtBot commented at 9:49 AM on December 10, 2018: member


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



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #14849 ([wip] depends: qt 5.9.7 by fanquake)
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.
MarcoFalke commented at 4:34 PM on December 10, 2018: member

ACK b7bee6af76e076e619b4ce5f165e81c6b4c52b2d
laanwj commented at 12:45 PM on December 12, 2018: member

ACK https://github.com/bitcoin/bitcoin/pull/14909/commits/b7bee6af76e076e619b4ce5f165e81c6b4c52b2d
laanwj merged this on Dec 12, 2018
laanwj closed this on Dec 12, 2018
laanwj referenced this in commit 96bde4f01f on Dec 12, 2018
fanquake deleted the branch on Dec 12, 2018
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#14725 qt: Bump the minimum Qt version to 5.2

doc: Update minimum required Qt #14909

Conflicts