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
chessx: fix build #46359
chessx: fix build #46359
Conversation
@GrahamcOfBorg build chessx |
Failure on x86_64-darwin (full log) Attempted: chessx Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: chessx Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: chessx Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: chessx Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: chessx Partial log (click to expand)
|
While you're at it, can you please limit this to |
@xeji done |
pkgs/games/chessx/default.nix
Outdated
''; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = http://chessx.sourceforge.net/; | ||
description = "ChessX allows you to browse and analyse chess games"; | ||
license = licenses.gpl2; | ||
maintainers = [maintainers.luispedro]; | ||
platform = platforms.linux; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
platforms
... eval error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ouch, good catch, thanks!
See https://hydra.nixos.org/build/80998335. Upstream doesn't support QT 5.11 ATM which broke compilation: ``` src/dialogs/savedialog.cpp: In constructor ‘SaveDialog::SaveDialog(QWidget*, Qt::WindowFlags)’: src/dialogs/savedialog.cpp:37:34: error: invalid use of incomplete type ‘class QButtonGroup’ group = new QButtonGroup(this); ``` The Arch community recommends to use an older QT version to fix this (https://aur.archlinux.org/packages/chessx/). Furthermore the `QT_PLUGIN_PATH` wasn't set properly which broke the runtime since QT coudln't find the `xcb` plugin: ``` qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in "" This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem. ``` Finally, some minor style fixes were made for consistent indentation. Addresses NixOS#45960
Success on aarch64-linux (full log) Attempted: chessx Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: chessx Partial log (click to expand)
|
I see a chess board and can make a move, so this looks great. Thank you! |
See https://hydra.nixos.org/build/80998335. Upstream doesn't support QT 5.11 ATM which broke compilation: ``` src/dialogs/savedialog.cpp: In constructor ‘SaveDialog::SaveDialog(QWidget*, Qt::WindowFlags)’: src/dialogs/savedialog.cpp:37:34: error: invalid use of incomplete type ‘class QButtonGroup’ group = new QButtonGroup(this); ``` The Arch community recommends to use an older QT version to fix this (https://aur.archlinux.org/packages/chessx/). Furthermore the `QT_PLUGIN_PATH` wasn't set properly which broke the runtime since QT coudln't find the `xcb` plugin: ``` qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in "" This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem. ``` Finally, some minor style fixes were made for consistent indentation. Addresses #45960 (cherry picked from commit 37032db)
backported: 3be4b8a |
Motivation for this change
See https://hydra.nixos.org/build/80998335.
Upstream doesn't support QT 5.11 ATM which broke compilation:
The Arch community recommends to use an older QT version to fix
this (https://aur.archlinux.org/packages/chessx/).
Furthermore the
QT_PLUGIN_PATH
wasn't set properly which broke theruntime since QT coudln't find the
xcb
plugin:Finally, some minor style fixes were made for consistent indentation.
Addresses #45960
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)