Project

General

Profile

Actions

Issue #10296

closed

Build Qt on all platforms

Added by spycrab0 over 7 years ago. Updated over 6 years ago.

Status:
Fixed
Priority:
Low
Assignee:
-
Target version:
-
% Done:

0%


Description

I suggest that the Qt Frontend should be built on all platforms, not only Windows.

Actions #1

Updated by ligfx over 7 years ago

Three buildbots need Qt installed:

One buildbot might need Qt updated:

  • FreeBSD buildbot: Currently has Qt 5.4.1 installed. FreshPorts lists Qt 5.7.1.

There's big question here: what version of Qt do we require? Currently, DolphinQt won't compile on anything less than 5.6:

  • MenuBar.cpp uses compile-time type-safety features added in Qt 5.6. This is the biggest issue IMO. The functionality can be implemented manually in a couple lines if needed, though ofc that's ugly.
  • The attribute AA_EnableHighDpiScaling was added in 5.6. This could be made optional with a simple #if QT_VERSION >= QT_VERSION_CHECK(5, 6, 0). Not compiling with this would change how DolphinQt2 looks on X11 on high-dpi displays, according to the docs.
  • GameFile.cpp uses QDataStream for caching, and explicitly requests the Qt 5.5 binary format.
Actions #2

Updated by blubberdiblub over 7 years ago

Debian and Ubuntu will also need qtbase5-private-dev since one recent PR.

Actions #3

Updated by spycrab0 over 7 years ago

Update

Dependencies should be present on all buildbots now.
Also: GameFile doesn't require 5.5 anymore.

Actions #4

Updated by ligfx over 7 years ago

Looks like the Debian buildbot got Qt5 installed, but the Ubuntu and macOS buildbots still don't (or maybe it's not configured correctly?).

Actions #5

Updated by spycrab0 almost 7 years ago

Looks like Qt is built everywhere now. The only remaining problem we have to tackle before we can close this issue is that DolphinQt2.app is currently not part of the OSX .dmg image

Actions #6

Updated by JosJuice over 6 years ago

  • Status changed from New to Fixed

This was fixed some time ago.

Actions

Also available in: Atom PDF