Skip to content
This repository has been archived by the owner on Mar 19, 2021. It is now read-only.

gui/src/servericonwidget.cpp: add missing import of QPainterPath #254

Merged
merged 1 commit into from
Jun 3, 2020

Conversation

invidian
Copy link
Contributor

@invidian invidian commented Jun 3, 2020

Otherwise building fails with the following error:

error: aggregate ‘QPainterPath path’ has incomplete type and cannot be
defined

Closes #253

Signed-off-by: Mateusz Gozdek mgozdekof@gmail.com

Otherwise building fails with the following error:

error: aggregate ‘QPainterPath path’ has incomplete type and cannot be
defined

Closes thestr4ng3r#253

Signed-off-by: Mateusz Gozdek <mgozdekof@gmail.com>
Copy link
Owner

@thestr4ng3r thestr4ng3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somehow can't reproduce myself with updated Arch a few minutes ago. But the fix is correct.

@thestr4ng3r thestr4ng3r merged commit 74e0f33 into thestr4ng3r:master Jun 3, 2020
@invidian
Copy link
Contributor Author

invidian commented Jun 3, 2020

Somehow can't reproduce myself with updated Arch a few minutes ago. But the fix is correct.

I didn't look too deeply into it, https://stackoverflow.com/a/9151602/2974814 led me to it and adding it seemed to help in my case, so decided to submit a patch. Thanks!

As far as I see, CI use QT 5.12 and Arch updated to QT 5.15 on 27th of May, maybe something changed.

@invidian invidian deleted the fix-building branch June 3, 2020 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to build: aggregate ‘QPainterPath path’ has incomplete type and cannot be defined
2 participants