Skip to content

Commit

Permalink
wasm: fix qpa plugin build system integration
Browse files Browse the repository at this point in the history
fix the plugin name (it was missing the leading 'q') and the name used
in configure (the latter making it unnecessary to mess with it in the
mkspec). the qt.prf override which forced linkage of the plugin is also
removed due to being completely redundant.

Change-Id: I94687a34a295c36754e36a298af902b656ba2ecc
Reviewed-by: Kyle Edwards <[email protected]>
Reviewed-by: Lorn Potter <[email protected]>
  • Loading branch information
ossilator committed Nov 21, 2018
1 parent f5f46aa commit 54dabdd
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 17 deletions.
12 changes: 0 additions & 12 deletions mkspecs/features/wasm/qt.prf

This file was deleted.

3 changes: 0 additions & 3 deletions mkspecs/wasm-emscripten/qmake.conf
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,4 @@ QMAKE_EXTENSION_STATICLIB = a # llvm bitcode
QMAKE_AR = emar cqs
QMAKE_DISTCLEAN += *.html *.js *.wasm

QT_QPA_DEFAULT_PLATFORM = wasm

QTPLUGIN.platforms = wasm
load(qt_config)
2 changes: 1 addition & 1 deletion src/gui/configure.pri
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ defineTest(qtConfTest_qpaDefaultPlatform) {
else: qnx: name = qnx
else: integrity: name = integrityfb
else: haiku: name = haiku
else: wasm: name = webassembly
else: wasm: name = wasm
else: name = xcb

$${1}.value = $$name
Expand Down
2 changes: 1 addition & 1 deletion src/plugins/platforms/wasm/wasm.pro
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
TARGET = wasm
TARGET = qwasm
CONFIG += static plugin
QT += \
core-private gui-private \
Expand Down

0 comments on commit 54dabdd

Please sign in to comment.